silverstripe-big-o/sapphire — comment on commit 929182f818cbed84c9fcd95ea53c6be395e4f9ea
File:  line 0
By chillu:

- The +10/-10 buttons don't disable properly
- Needs unit tests, but I suppose you know that, and just committed something to get Jeremy onto styling :)
- The concept of "ten pages forward/backward" should be less hardcoded in comments and variable names. Ideally its configureable, but at the least it should be one variable in the local scope, as opposed to all over the place. Its also not accurate, on the second page the "backward" link will just skip 1 page back, not 10. How about calling it a "skip link", and the actions "action_skip_prev"/"action_skip_next"?