Closed
Bug 1261778
Opened 9 years ago
Closed 6 years ago
[RTL] The left and right keyboard navigation of tree nodes are wrong direction in RTL locales
Categories
(Toolkit :: UI Widgets, defect)
Toolkit
UI Widgets
Tracking
()
VERIFIED
FIXED
mozilla71
People
(Reporter: magicp.jp, Assigned: itiel_yn8)
References
()
Details
(Keywords: rtl)
Attachments
(1 file)
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Firefox/45.0
Build ID: 20160315153207
Steps to reproduce:
1. Start any versions Firefox with RTL locales (e.g. arabic)
2. Go to any tree nodes (e.g. Bookmarks sidebar)
3. Hit the left and right key on a collapsed node
Actual results:
Left key: RTL tree node (twisty) is closed.
Right key: RTL tree node is opened.
Expected results:
Left key: RTL tree node (twisty) should be opened.
Right key: RTL tree node should be closed.
Has STR: --- → yes
OS: Unspecified → All
Hardware: Unspecified → All
Comment 1•9 years ago
|
||
Should this perhaps be a XUL issue? Are regular web pages able to do such navigation without it being custom JS handlers?
Ehsan, do you have RTL thoughts (sorry to bug you)?
Flags: needinfo?(ehsan)
Comment 2•9 years ago
|
||
Yes, this is a XUL issue, more specifically a bug in the VK_LEFT/VK_RIGHT keydown handlers in tree.xml. Should be pretty simple to fix.
Component: Keyboard: Navigation → XUL Widgets
Flags: needinfo?(ehsan)
Product: Core → Toolkit
Comment 3•9 years ago
|
||
Split the devtools issue off to bug 1310055 since DevTools don't use the native XUL tree.
No longer blocks: dt-rtl
Keywords: checkin-needed
Pushed by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/60d2e1c34454
[RTL] Fix the left and right keyboard navigation direction of tree nodes r=bgrins
Keywords: checkin-needed
Comment 6•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•