Closed Bug 991072 Opened 11 years ago Closed 11 years ago

Zoom percentage label not aligned properly when placed in a toolbar

Categories

(Firefox :: Theme, defect)

Product:
Firefox
Component:
Theme
Version:
31 Branch
Platform:
All
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
Firefox 31
Tracking Flags:
Tracking Status
firefox29 --- verified
firefox30 --- verified
firefox31 --- verified

People

(Reporter: mikedeboer, Assigned: mikedeboer)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [Australis:P4])

Attachments

(1 file)

Patch v1: fix zoom percentage label to be centered in any toolbar
1.12 KB, patch
mconley
: review+
Sylvestre
: approval-mozilla-aurora+
Sylvestre
: approval-mozilla-beta+
Details | Diff | Splinter Review
After many attempts at fixing this before, the zoom percentage label doesn't get positioned in the middle when placed in the TabsToolbar, navbar or PersonalToolbar. Let's fix this once and for all.
Whiteboard: [Australis:P4]
Comment on attachment 8400649 [details] [diff] [review] Patch v1: fix zoom percentage label to be centered in any toolbar On second thought, Mike, you have easier access to Mac, I believe...
Attachment #8400649 - Flags: review?(jaws) → review?(mconley)
Comment on attachment 8400649 [details] [diff] [review] Patch v1: fix zoom percentage label to be centered in any toolbar Review of attachment 8400649 [details] [diff] [review]: ----------------------------------------------------------------- Looks good! :) Thanks Mike!
Attachment #8400649 - Flags: review?(mconley) → review+
Thanks! Pushed as https://hg.mozilla.org/integration/fx-team/rev/5ad244ee8032
status-firefox29: --- → affected
status-firefox30: --- → affected
status-firefox31: --- → affected
Whiteboard: [Australis:P4] → [Australis:P4][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/5ad244ee8032
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P4][fixed-in-fx-team] → [Australis:P4]
Target Milestone: --- → Firefox 31
Comment on attachment 8400649 [details] [diff] [review] Patch v1: fix zoom percentage label to be centered in any toolbar [Approval Request Comment] Bug caused by (feature/regressing bug #): Australis User impact if declined: When an OSX user puts the Zoom controls on a toolbar, he/ she will notice the zoom percentage label aligned too much toward the bottom of the button. This patches fixes that, regardless of the toolbar type. Testing completed (on m-c, etc.): landed on m-c. Risk to taking this patch (and alternatives if risky): minor String or IDL/UUID changes made by this patch: n/a
Attachment #8400649 - Flags: approval-mozilla-beta?
Attachment #8400649 - Flags: approval-mozilla-aurora?
Attachment #8400649 - Flags: approval-mozilla-beta?
Attachment #8400649 - Flags: approval-mozilla-beta+
Attachment #8400649 - Flags: approval-mozilla-aurora?
Attachment #8400649 - Flags: approval-mozilla-aurora+
User Agents: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Firefox/31.0 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:30.0) Gecko/20100101 Firefox/30.0 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:29.0) Gecko/20100101 Firefox/29.0 Verified this issue as fixed using: - latest Firefox Nightly (build ID: 20140407030203) - latest Firefox Aurora (build ID: 20140407004002) - Firefox 29 beta 6 (build ID: 20140407135746).
Status: RESOLVED → VERIFIED
status-firefox29: fixed → verified
status-firefox30: fixed → verified
status-firefox31: fixed → verified
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: