Closed
Bug 1407007
Opened 7 years ago
Closed 7 years ago
[RTL] Search icon badges on the search bar are aligned to the left instead of right
Categories
(Firefox :: General, defect, P1)
Firefox
General
Tracking
()
VERIFIED
FIXED
Firefox 58
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | --- | verified |
firefox58 | --- | verified |
People
(Reporter: itiel_yn8, Assigned: daleharvey)
References
Details
(Keywords: regression, Whiteboard: [reserve-photon-visual])
Attachments
(3 files)
936 bytes,
image/png
|
Details | |
1.49 KB,
image/png
|
Details | |
59 bytes,
text/x-review-board-request
|
dao
:
review+
ritu
:
approval-mozilla-beta+
|
Details |
This started a few days ago. See attached, the small arrow icon is overlapping the search icon and the green plus sign is on the wrong side of the icon. Previous implementation was okay in this regard. Also attached.
status-firefox58:
--- → affected
Comment 2•7 years ago
|
||
Dão or Johann, do you know what might have broken this? I recall there were a few changes with the search icons that could be responsible here?
Flags: needinfo?(jhofmann)
Flags: needinfo?(dao+bmo)
Whiteboard: [photon-visual][triage]
Comment 3•7 years ago
|
||
Sounds like bug 1403359. Dale, can you confirm? :)
Flags: needinfo?(jhofmann) → needinfo?(dharvey)
Assignee | ||
Comment 4•7 years ago
|
||
Yup, will take cheers
Assignee: nobody → dharvey
Flags: needinfo?(dharvey)
Flags: needinfo?(dao+bmo)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8916943 [details] Bug 1407007 - Fix RTL alignment of search icon. Not ready yet
Attachment #8916943 -
Flags: review?(dao+bmo)
Assignee | ||
Comment 7•7 years ago
|
||
Comment on attachment 8916943 [details] Bug 1407007 - Fix RTL alignment of search icon. Actual false alert, I think this is good now
Attachment #8916943 -
Flags: review?(dao+bmo)
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8916943 [details] Bug 1407007 - Fix RTL alignment of search icon. https://reviewboard.mozilla.org/r/187980/#review193142
Attachment #8916943 -
Flags: review?(dao+bmo) → review+
Pushed by dharvey@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2f5f7289c2d5 Fix RTL alignment of search icon. r=dao
Updated•7 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Updated•7 years ago
|
Flags: qe-verify+
Assignee | ||
Comment 10•7 years ago
|
||
Comment on attachment 8916943 [details] Bug 1407007 - Fix RTL alignment of search icon. Approval Request Comment [Feature/Bug causing the regression]: https://bugzilla.mozilla.org/show_bug.cgi?id=1403359 [User impact if declined]: RTL users will have visually incorrect icon [Is this code covered by automated tests?]: nope [Has the fix been verified in Nightly?]: [Needs manual test from QE? If yes, steps to reproduce]: [List of other uplifts needed for the feature/fix]: [Is the change risky?]: Nope [Why is the change risky/not risky?]: Updating missing CSS for RTL only [String changes made/needed]:
Attachment #8916943 -
Flags: approval-mozilla-beta?
Comment 11•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2f5f7289c2d5
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
status-firefox57:
--- → affected
Comment on attachment 8916943 [details] Bug 1407007 - Fix RTL alignment of search icon. Recent regression related to photon changes, Beta57+
Attachment #8916943 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 13•7 years ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/0aac8b210514c48af1ca90957d6a2216407b66ba
Reporter | ||
Comment 14•7 years ago
|
||
Verified fixed on latest Nightly.
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
status-firefox56:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Comment 15•7 years ago
|
||
I tried to recreate this bug using Nightly 58 from 2017-10-09 on WIndows 10x64, Mac OS X 10.11 and Ubuntu 16.04x64, but I couldn't reproduce it. The icon that I was inspecting was from the search bar (I added it back on the toolbar). But the badges were always aligned to the right side of the icon. Is there something that I missed? Or can you please give me additional steps to reproduce this bug?
Assignee | ||
Comment 16•7 years ago
|
||
You need to set intl.uidirection to 1 to trigger RTL mode, with that and before this patch the search icon will be as in the screenshot
Comment 17•7 years ago
|
||
Thank you, Dale, for the information. It was really helpful. I managed to reproduce the bug using Nightly 58 from 2017-10-09 on Windows 10x64, Mac OS X 10.11 and Ubuntu 16.04x64. I retested everything using beta 57.0b8 on the same platforms and the bug is not reproducing anymore.
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•