Closed Bug 1014698 Opened 11 years ago Closed 10 years ago

Unable to see the keyboard focus indicator on the category list of in-content preferences

Categories

(Firefox :: Settings UI, defect)

All
Windows 8.1
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 33

People

(Reporter: jaws, Assigned: peregrino)

References

Details

Attachments

(1 file, 2 obsolete files)

Cycling through Tab to get focus on the category list, it is impossible to know when focus is on that list. However, when it is on the list, pressing up and down will cycle through the categories. We need to make keyboard focus be more obvious for the category list.
I was looking at this one last night, and got to a point where I could apply styles to the category list for when it's focused. But then another problem appeared: when clicking on a category with the mouse, the :-moz-focusring styles stay applied until you click somewhere else and move the focus away. Do anyone have a suggestion to fix that?
Attached patch WIP patch (obsolete) (deleted) — Splinter Review
This is a work in progress, it has the problems mentioned in my earlier comment
(In reply to Hernán Rodriguez Colmeiro (:peregrino) from comment #1) > I was looking at this one last night, and got to a point where I could apply > styles to the category list for when it's focused. > But then another problem appeared: when clicking on a category with the > mouse, the :-moz-focusring styles stay applied until you click somewhere > else and move the focus away. > > Do anyone have a suggestion to fix that? You could add keydown and mousedown listeners that toggle an attribute to show the focus indicators: categories.addEventListener("keydown", function() { this.setAttribute("keyboard-navigation", "true"); }); categories.addEventListener("mousedown", function() { this.removeAttribute("keyboard-navigation"); }); Then in the CSS, #categories[keyboard-navigation="true"]:-moz-focusring > .category[current] { border-top: 1px #ffffff dotted; border-bottom: 1px #ffffff dotted; }
Assignee: nobody → colmeiro
Status: NEW → ASSIGNED
Flags: needinfo?(colmeiro)
Attached patch bug-1014698.diff (obsolete) (deleted) — Splinter Review
Updated patch. This one fixes the problem.
Attachment #8430185 - Attachment is obsolete: true
Attachment #8433551 - Flags: review?(jaws)
Flags: needinfo?(colmeiro)
Comment on attachment 8433551 [details] [diff] [review] bug-1014698.diff Review of attachment 8433551 [details] [diff] [review]: ----------------------------------------------------------------- Looks and works good. Thanks!
Attachment #8433551 - Flags: review?(jaws) → review+
Do you happen to have a Try link handy by chance? :)
Keywords: checkin-needed
Flags: needinfo?(colmeiro)
Uhm. I used to have L1 commit access on this email account but I think got deactivated by a prolonged inactivity I had. Is there a chance to reactivate it so I can push to try?
Flags: needinfo?(colmeiro)
(In reply to Hernán Rodriguez Colmeiro (:peregrino) from comment #7) > Uhm. I used to have L1 commit access on this email account but I think got > deactivated by a prolonged inactivity I had. Is there a chance to reactivate > it so I can push to try? Please file a new bug like https://bugzilla.mozilla.org/show_bug.cgi?id=664170 to request reactivation of your level 1 privleges.
I just saw that this happened before and the threshold of inactivity is 3months. I won't bother people to reactivate my L1 until I'm sure I'll be able to not let it expire again ;)
Attachment #8433551 - Attachment is obsolete: true
Assignee: colmeiro → gijskruitbosch+bugs
Ugh, sorry. /me slaps bzexport
Assignee: gijskruitbosch+bugs → colmeiro
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 33
Verified fixed on Windows 8.1 64bit using latest Nightly 34.0a1 (buildID: 20140729030202).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: