Closed Bug 1594825 Opened 5 years ago Closed 4 years ago

[RTL] Keyboard navigation between trackers/fingerprinters/cryptominers is reversed in about:protections

Categories

(Firefox :: Protections UI, defect, P2)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1603156

People

(Reporter: itiel_yn8, Unassigned)

Details

(Keywords: rtl)

Attachments

(1 obsolete file)

STR:

  1. Change intl.uidirection to 1 in about:config
  2. Open about:protections
  3. Click any of the tracker types
  4. Try to navigate between them using the arrow keys

AR:
Keyboard navigation is in reverse.

ER:
Correct keyboard navigation.

do you have time to look at this?

Flags: needinfo?(nhnt11)

Sorry for the delayed response. I don't think I have time for this at the moment, I'm not familiar enough with the about:protections code to pick this up quickly.

Flags: needinfo?(nhnt11)

The priority flag is not set for this bug.
:ewright, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(ewright)
Flags: needinfo?(ewright)
Priority: -- → P2
Assignee: nobody → ewright
Status: NEW → ASSIGNED

On further inspection I believe this is a Firefox bug. The patch I posted essentially lifts the elements off the page, then re-inserts them, this seems to reset something and fix the navigation, but i have no idea why. However, I've also noticed backwards navigation if you simply add this to a page:

<body dir="rtl">
    <form>
    <div>
    <input type="radio" name="color" value="yellow" checked> 1
    <input type="radio" name="color" value="red"> 2
    <input type="radio" name="color" value="green"> 3 
    <input type="radio" name="color" value="pink"> 4
    </div>
    </form> 
</body>

There is also backwards navigation in our upper nav bar icons. For example, in rtl tab to some addon icons, then push the arrow keys to cycle through them.

I'll keep this bug open as we may be able to find a temporary workaround for this for just the protection report, and file new ones for radio buttons in general and toolbaritems

See Also: → 1603156
Attachment #9113560 - Attachment is obsolete: true
Assignee: ewright → nobody
Status: ASSIGNED → NEW

Closing as duplicate.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
See Also: 1603156
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: