Adjust the vertical padding of the identity box across platforms

RESOLVED FIXED in Firefox 42

Status

()

defect
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: Paolo, Assigned: Paolo)

Tracking

({regression})

Trunk
Firefox 42
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The inner vertical padding of the identity box is too small, causing the height of the address bar on Windows and Linux to be different if there are no other elements on the toolbar.
(Assignee)

Comment 1

4 years ago
Bug 1192434 - Adjust the vertical padding of the identity box across platforms. r=bgrins
Attachment #8645224 - Flags: review?(bgrinstead)
(Assignee)

Updated

4 years ago
Assignee: nobody → paolo.mozmail
Status: NEW → ASSIGNED
Comment on attachment 8645224 [details]
MozReview Request: Bug 1192434 - Adjust the vertical padding of the identity box across platforms. r=bgrins

https://reviewboard.mozilla.org/r/15443/#review13831

In OSX the location bar in a popup window moves down a px or so when being focused with this patch applied
Attachment #8645224 - Flags: review?(bgrinstead)
(Assignee)

Comment 3

4 years ago
Comment on attachment 8645224 [details]
MozReview Request: Bug 1192434 - Adjust the vertical padding of the identity box across platforms. r=bgrins

Bug 1192434 - Adjust the vertical padding of the identity box across platforms. r=bgrins
Attachment #8645224 - Flags: review?(bgrinstead)
Comment on attachment 8645224 [details]
MozReview Request: Bug 1192434 - Adjust the vertical padding of the identity box across platforms. r=bgrins

https://reviewboard.mozilla.org/r/15443/#review13847

I've tested this on Windows and OSX and it seems to work fine, and does cause popup windows nav height to match normal windows.  As far as I understand, this is restoring the margin size previous to Bug 1185362 so it shouldn't cause any problems
Attachment #8645224 - Flags: review?(bgrinstead) → review+
https://hg.mozilla.org/mozilla-central/rev/ce72fc6f687a
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42

Updated

3 years ago
Depends on: 1292877
You need to log in before you can comment on or make changes to this bug.