Posted by hansamurai on December 13, 2009 at 6:11am
5 followers
| Project: | Custom Pagers |
| Version: | 6.x-1.x-dev |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
When there are more than 100 nodes represented in the pager, I noticed that the text of the pager would look like this: "previous100 of 100 next". Due to the use of fixed width margins in the CSS, I propose this solution to drop the fixed width and instead use margins. It now looks appropriately like "previous 100 of 100 next".
Attached is the patch and before and after screenshots.
This is my first proposed patch, so let me know if I'm missing something.
| Attachment | Size |
|---|---|
| custom_pagers_css_width.patch | 467 bytes |
| before.png | 3.84 KB |
| after.png | 4.22 KB |
Comments
#1
Committed, thanks!
#2
Automatically closed -- issue fixed for 2 weeks with no activity.
#3
I don't quite see how this will work for everyone. Having updated the module the overall layout of the pager is squashed to the left with the Barron theme because I don't think width: auto cuts much ice with floats. Please see before and after pics.
#4
"Auto" setting fixes the initial display for most sites, and makes theming less difficult.