Closed
Bug 1389921
Opened 7 years ago
Closed 7 years ago
New Tab button should move 2px to the left
Categories
(Firefox :: Theme, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | --- | verified |
People
(Reporter: Virtual, Assigned: dao)
References
Details
(Keywords: nightly-community, ux-consistency, Whiteboard: [reserve-photon-visual])
Attachments
(7 files)
STR:
1. Hover over "Open a new tab" button in tab bar
and see that it need some tuning to be like on Photon specification - http://design.firefox.com/people/shorlander/photon/Mockups/windows-7.html
Some data
"Open a new tab" button size:
36 x 26 px - before landing patch from bug #1377184
28 x 28 px - after landing patch from bug #1377184
28 x 28 px - in Photon specification
"Open a new tab" button active hover area:
36 x 26 px - before landing patch from bug #1377184
36 x 26 px - after landing patch from bug #1377184
28 x 28 px - in Photon specification
Space between Tab and "Open a new tab" button:
4 px - after landing patch from bug #1377184
2 px - in Photon specification
So in tl;dr "Open a new tab" button:
- should be moved 2 px to the left
- active hover area should be only on the button area
Regression range pushlog:
https://hg.mozilla.org/integration/autoland/json-pushes?changeset=500a01cf896dbb9883cb85f162552e20b8facfc0&full=1
Caused by:
Bug 1377184 - Use new Photon hover effect also on "Show more bookmarks" and extensions buttons icons in Bookmarks Toolbar
Reporter | ||
Updated•7 years ago
|
Reporter | ||
Updated•7 years ago
|
Keywords: regression → ux-consistency
One more thing. Looks like the "Open a new tab" button has slightly different parameters for every UI density option. For example there is only 1 px gap between Tab and Open a new tab button in Compact mode.
Reporter | ||
Comment 8•7 years ago
|
||
I forgot to mention, that my screenshots are with Compact density mode.
As far as I can see all your screenshots are from Normal density mode. Check the density options on the Mockup page. There is a difference in Back button design.
Reporter | ||
Comment 10•7 years ago
|
||
Ops, yes, you're right.
Assignee | ||
Comment 11•7 years ago
|
||
(In reply to Virtual_ManPL [:Virtual] - (please needinfo? me - so I will see your comment/reply/question/etc.) from comment #0)
> So in tl;dr "Open a new tab" button:
> - should be moved 2 px to the left
> - active hover area should be only on the button area
We don't have plans to do the latter. Morphing bug.
Summary: Tune "Open a new tab" button → New Tab button should move 2px to the left
Whiteboard: [photon-visual] [triage] → [reserve-photon-visual]
Updated•7 years ago
|
Flags: qe-verify?
Priority: -- → P4
Assignee | ||
Updated•7 years ago
|
Priority: P4 → P3
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → dao+bmo
Flags: qe-verify? → qe-verify+
Priority: P3 → P1
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Status: NEW → ASSIGNED
Iteration: --- → 57.2 - Aug 29
Comment 14•7 years ago
|
||
mozreview-review |
Comment on attachment 8898661 [details]
Bug 1389921 - Stop setting a width on the new tab button to remove excess padding around it.
https://reviewboard.mozilla.org/r/170042/#review175932
Looks good cheers
Attachment #8898661 -
Flags: review?(dharvey) → review+
Comment 15•7 years ago
|
||
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cc018eee995b
Stop setting a width on the new tab button to remove excess padding around it. r=daleharvey
Comment 16•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Just for the record, this caused some conflicts when I merged bug 1352119 to m-c from inbound. I think I resolved it correctly, though.
Flags: needinfo?(dao+bmo)
Assignee | ||
Comment 18•7 years ago
|
||
(In reply to Wes Kocher (:KWierso) from comment #17)
> Just for the record, this caused some conflicts when I merged bug 1352119 to
> m-c from inbound. I think I resolved it correctly, though.
Looks like it.
Flags: needinfo?(dao+bmo)
Reporter | ||
Comment 19•7 years ago
|
||
I'm confirming that bug is fixed, starting in Mozilla Firefox Nightly 57.0a1 (2017-08-22), so I'm marking this bug as VERIFIED. Thanks.
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
Flags: qe-verify+
Comment 20•7 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•