Closed
Bug 1274749
Opened 8 years ago
Closed 8 years ago
Useless one free space between dropmarker and separator in address bar
Categories
(Firefox :: Theme, defect)
Tracking
()
VERIFIED
FIXED
Firefox 49
Tracking | Status | |
---|---|---|
firefox48 | --- | unaffected |
firefox49 | + | verified |
People
(Reporter: Virtual, Assigned: baku)
References
Details
(4 keywords)
Attachments
(3 files, 1 obsolete file)
Opening new window in Nightly will get dropmarker moved one place to the left of its normal position, opening new tab fixes the issue in this window. [Tracking Requested - why for this release]: Regression
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Comment 3•8 years ago
|
||
Regression window (mozilla-central) Good: https://ftp.mozilla.org/pub/firefox/nightly/2016/05/2016-05-17-03-02-11-mozilla-central/ Bad: https://ftp.mozilla.org/pub/firefox/nightly/2016/05/2016-05-18-03-02-34-mozilla-central/ Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=a884b96685aa13b65601feddb24e5f85ba861561&tochange=f3f2fa1d7eed5a8262f6401ef18ff8117a3ce43e Probably caused by: 9ffc0498ad7e Andrea Marchesini — Bug 1267923 - Move the CSS rules to ContextualIdentityService, r=mconley
Blocks: 1267923
Has Regression Range: --- → yes
Has STR: --- → yes
Flags: needinfo?(amarchesini)
Keywords: regressionwindow-wanted
Assignee | ||
Comment 4•8 years ago
|
||
Flags: needinfo?(amarchesini)
Attachment #8755220 -
Flags: review?(gijskruitbosch+bugs)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → amarchesini
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•8 years ago
|
Status: NEW → ASSIGNED
Comment 5•8 years ago
|
||
Comment on attachment 8755220 [details] [diff] [review] fix.patch I can reproduce this on inbound tip from a few minutes ago, but this patch does not apply there, so I don't know how to review this / what this is supposed to be doing (and, if the code you're changing here isn't there yet, it feels like it also won't fix the issue that we're dealing with already on Nightly).
Attachment #8755220 -
Flags: review?(gijskruitbosch+bugs)
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8755220 -
Attachment is obsolete: true
Attachment #8755396 -
Flags: review?(gijskruitbosch+bugs)
Updated•8 years ago
|
Attachment #8755396 -
Flags: review?(gijskruitbosch+bugs) → review+
Comment 10•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c2954f2b8ab7
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•8 years ago
|
Status: RESOLVED → VERIFIED
Comment 11•8 years ago
|
||
Why is this element not just hidden by default, i.e. <hbox id="userContext-icons" hidden="true">?
Flags: needinfo?(amarchesini)
Assignee | ||
Comment 12•8 years ago
|
||
Dao, you are right. Can you file a follow up?
Flags: needinfo?(amarchesini)
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•8 years ago
|
Blocks: 1276585
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Comment 13•8 years ago
|
||
(In reply to Andrea Marchesini (:baku) from comment #12) > Dao, you are right. Can you file a follow up? Done. It's in bug #1276585.
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•8 years ago
|
No longer blocks: 1276585
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
Keywords: nightly-community
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
QA Contact: Virtual
You need to log in
before you can comment on or make changes to this bug.
Description
•