Download & Extend

"Previous" pager button too narrow

Project:Omega
Version:7.x-3.x-dev
Component:User interface
Category:bug report
Priority:normal
Assigned:gilcrespo
Status:closed (duplicate)
Issue tags:CSS, omega, patch

Issue Summary

When using a multi-page view, with a pager, In version 7.x-3.0, the previous pager said "prev". Now, in 7.x-3.1 it says "previous", but the button seems too narrow to accommodate the word previous and the < chevron, so they get split onto two lines. I don't think I'm doing anything special with the fonts. It appears to be doing the same thing in all the browsers and platforms I've tested (Safari, Chrome, FF, IE8, IE9, OSX, Win, iOS)

Proposed resolution: make the button slightly wider, or revert to using "prev"

AttachmentSizeStatusTest resultOperations
Screen Shot 2012-04-02 at 11.20.50 AM.png7.33 KBIgnored: Check issue status.NoneNone

Comments

#1

I'm having the same problem. I was going to hack the omega-visuals.css to change the width to 70px, but decided to add the following to my subtheme's default css file instead, which fixed my problem:

/* hack to fix previous pager button from omega-visuals.css */
.pager li.pager-previous a {
width: 70px;
}

It widens the Previous button.
Question: Is the "Previous" text inherited from Views?

I look forward to a real fix in the Views module and/or the Omega theme.

#2

I came up with a similar solution and decided to see if anyone else had filed an issue.

For now I'm using the following in my theme's global.css file and it seems to work well on mobile through desktop sizes

li.pager-previous a.active {
    width: 70px;
}

#3

Doing the same css hack, works as expected afterwords.

#4

Assigned to:Anonymous» philosurfer
Status:active» patch (to be ported)

here is a patch to be ported. Changed it to 65px. 70 was too wide.... thoughts?

AttachmentSizeStatusTest resultOperations
omega-pager_css_fix-1512454.patch344 bytesIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch omega-pager_css_fix-1512454.patch. Unable to apply patch. See the log in the details link for more information.View details | Re-test

#5

I changed mine to 65px and it is fine with me.

#6

Status:patch (to be ported)» needs review

#7

Status:needs review» needs work

The last submitted patch, omega-pager_css_fix-1512454.patch, failed testing.

#8

Status:needs work» needs review

#4: omega-pager_css_fix-1512454.patch queued for re-testing.

#9

Status:needs review» needs work

The last submitted patch, omega-pager_css_fix-1512454.patch, failed testing.

#10

Renaming the name of patch.

AttachmentSizeStatusTest resultOperations
omega-pager-css-fix-1512454.patch344 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 0 pass(es).View details | Re-test

#11

Priority:minor» normal
Assigned to:philosurfer» gilcrespo
Status:needs work» needs review

This patch omega-pager_css_fix-1512454.patch was working on my machine. I just renamed following the standards.

Also i changed the priority to maybe get attention.

Thanks

#12

Status:needs review» needs work

The last submitted patch, omega-pager-css-fix-1512454.patch, failed testing.

#13

Version:7.x-3.1» 7.x-3.x-dev
Status:needs work» needs review

Patches always need to be applied to the branch, not a specific version. Changing to 7.x-3.x-dev

#14

#10: omega-pager-css-fix-1512454.patch queued for re-testing.

#15

Status:needs review» closed (duplicate)

#10 is very bad, fixed width
and this is a duplicate of #1447146: pager layout not flexible
better patches there too.

nobody click here