Closed
Bug 1367015
Opened 7 years ago
Closed 7 years ago
Use back.svg for the back button and remove back-large.svg
Categories
(Firefox :: Theme, defect, P1)
Firefox
Theme
Tracking
()
People
(Reporter: johannh, Assigned: dao)
References
Details
(Whiteboard: [photon-visual][p1][57])
Attachments
(1 file)
The tail part of the back button icon is a bit longer than it previously was. See https://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=4b8939ed128172fcd3433ee25f336a8c1ba77ab0&newProject=mozilla-central&newRev=2b6f6881a24a726364054aa69031927e2f3fc5fa This makes the back and forward buttons look different, which is definitely not good for the Photon compact mode but I guess it's also a general visual regression we should fix.
Flags: qe-verify+
Updated•7 years ago
|
Priority: P1 → P2
QA Contact: brindusa.tot
Assignee | ||
Updated•7 years ago
|
status-firefox54:
unaffected → ---
status-firefox55:
affected → ---
Summary: Back button icon slightly stretched → Use back.svg for the back button and remove back-large.svg
Whiteboard: [photon-visual][p2] → [photon-visual][p1][57]
Assignee | ||
Updated•7 years ago
|
Keywords: regression
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Priority: P2 → P1
Updated•7 years ago
|
Iteration: --- → 55.6 - May 29
Reporter | ||
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8870395 [details] Bug 1367015 - Use back.svg for the back button and remove back-large.svg. https://reviewboard.mozilla.org/r/141848/#review145530 Thanks!
Attachment #8870395 -
Flags: review?(jhofmann) → review+
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/377891d444ed Use back.svg for the back button and remove back-large.svg. r=johannh
Comment 4•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/377891d444ed
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Reporter | ||
Comment 5•7 years ago
|
||
Screenshots: https://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=1ce55499e8193d53aeff91546b16e0c580ebd6f1&newProject=mozilla-central&newRev=545ffce30eac33921dcb44b5a916c78c5cb0fc95
Comment 6•7 years ago
|
||
I verified this issue on Windows 7, Windows 10, Ubuntu 16.04 with Nightly 57.0a1 (2017-08-08) and I can confirm that the back button looks like the one from https://mozilla.invisionapp.com/share/ENBBK0F9U#/screens/237801528. The screenshots from comment 5 are not the same as the ones from specs. I will mark this as verified based on the fact that the actual results are the same as the specs.
You need to log in
before you can comment on or make changes to this bug.
Description
•