Created attachment 4912 [details] Screenshot of the proposed revised interface I propose a change to the style of the pager which has been added to some tables sorted by the jQuery tablesorter. See the attached image. This includes a change proposed in comments on Bug 6616 to make it clear that the pages notation (1/3) cannot be directly edited. This bug cannot be tested without the patch for Bug 6616.
Created attachment 5751 [details] [review] Proposed fix Adds some style to the table pager include. Also styles the pages "1/2" input field so that it does not look like an input field (since editing the value does nothing).
Created attachment 5753 [details] [review] [SIGNED-OFF] Fix for Bug 6689 - Improve styling of table pager Adds some style to the table pager include. Also styles the pages "1/2" input field so that it does not look like an input field (since editing the value does nothing). Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
CSS-only fix, with a very attractive screenshot attached. Depends on 6616, but as we're just waiting on that to be committed, I'll also mark this as Passed QA (happily).
Patch pushed, please test
Created attachment 12251 [details] [review] Bug 6689 [Follow-up] Improve styling of table pager Small correction to add non-vendor-prefixed border radius declaration to make style consistent on newer browsers. To test, view a page with a sorted table and pager controls, like members/readingrec.pl. The pager controls "box" should show rounded corners in Firefox 15+
Created attachment 12295 [details] [review] [SIGNED-OFF] Bug 6689 [Follow-up] Improve styling of table pager Small correction to add non-vendor-prefixed border radius declaration to make style consistent on newer browsers. To test, view a page with a sorted table and pager controls, like members/readingrec.pl. The pager controls "box" should show rounded corners in Firefox 15+ Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
passing QA, trivial patch...
Patch pushed to master