Closed Bug 1317268 Opened 5 years ago Closed 5 years ago

Fix unicode bidi for bookmarks/history/... label of tooltip in RTL locales

Categories

(Firefox :: Bookmarks & History, defect)

Product:
Firefox
Component:
Bookmarks & History
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
Firefox 53
Tracking Flags:
Tracking Status
firefox53 --- verified

People

(Reporter: Gijs, Assigned: Gijs)

References

Details

Attachments

(1 file)

Bug 1317268 - fix direction of label in bookmarks/history tooltip in RTL,
58 bytes, text/x-review-board-request
mak
: review+
Details 
+++ This bug was initially created as a clone of Bug #1313751 +++

Steps to Reproduce:
1. Start any versions Firefox in RTL locales (e.g. Arabic)
2. Open "https://ftp.mozilla.org/pub/firefox/releases/48.0/mac/"
3. Open the history sidebar (ctrl-h, or cmd-shift-h on osx)
4. click "View" and then "By Last Visited"
5. Mouse hover over the item for https://ftp.mozilla.org/pub/firefox/releases/48.0/mac/ (should be all the way at the top)

Actual Results:
the label reads:
"/Directory Listing: /pub/firefox/releases/48.0/mac"
 ^
Expected Results:
"Directory Listing: /pub/firefox/releases/48.0/mac/"
                                                  ^
Comment on attachment 8810344 [details]
Bug 1317268 - fix direction of label in bookmarks/history tooltip in RTL,

https://reviewboard.mozilla.org/r/92722/#review93106
Attachment #8810344 - Flags: review?(mak77) → review+
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1bd4e2805bec
fix direction of label in bookmarks/history tooltip in RTL, r=mak
https://hg.mozilla.org/mozilla-central/rev/1bd4e2805bec
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
I have reproduced the bug in Nightly 52.0a1 (2016-11-14) with the instruction from comment 0 and Linux 64 Bit

Bug is fixed now on Latest Beta 53.0b3 (Build ID:20170316115347)
User Agent:Mozilla/5.0 (X11; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0

[testday-20170317]
Thanks!
Status: RESOLVED → VERIFIED
status-firefox53: fixed → verified
You need to log in before you can comment on or make changes to this bug.