Focus styling isn't visible on trees in some scenarios (tabbing to the tree doesn't highlight the first item, navigating items with Ctrl+Up/Down)
Categories
(Toolkit :: XUL Widgets, defect, P3)
Tracking
()
People
(Reporter: u601362, Assigned: john)
References
(Regression)
Details
(Keywords: access, regression)
Attachments
(1 file, 2 obsolete files)
(deleted),
text/x-phabricator-request
|
Details |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Firefox/68.0
Steps to reproduce:
Open Firefox, browse to any page.
Press Ctrl+H (opens history), place cursor into the history search field, press tab key on keyboard two times.
Actual results:
Selected item is "today". But UI does not highlight this item
Expected results:
Either a dotted line or colorized background should indicate which is the focused item.
Updated•5 years ago
|
Comment 1•5 years ago
|
||
It looks like the focus isn't set up correctly somehow. Opening the sidebar and tabbing to the tree seems to give focus to the entire tree, but the first item is not selected.
You can press left or right-arrow to expand it. Pressing up or down arrows gives the expected highlight.
This also affects the bookmarks sidebar as well as the history one.
Comment 2•5 years ago
|
||
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=98d17facfe0093c4120cedd656d82b46822c1964&tochange=c8c488989a62c9ad8805569062c89ff74e641418
Regressed by: c8c488989a62c9ad8805569062c89ff74e641418 Tim Nguyen — Bug 1469287 - Implement new shared tree styling. r=dao
Before landing Bug 1469287, at least focus outline is indicated for the first treechildren.
Updated•5 years ago
|
Too late for 70 but we could still take a patch in 71.
Comment 5•5 years ago
|
||
It seems like this is a general problem with trees based on Bug 1616843. Another STR from https://bugzilla.mozilla.org/show_bug.cgi?id=1616843#c0:
Steps to reproduce:
Trying to navigate in a tree (like bookmarks view) with keyboard, and moving around without selection (ie. with <ctrl>up/down).
Actual results:
No visual feedback except that the scroll area may move if we go far enough or if we open a sub-tree.
Expected results:
We should see something like the usual dotted-line outline.
Comment 6•5 years ago
|
||
Tim, do you have any idea which rule removed in https://hg.mozilla.org/mozilla-central/rev/c8c488989a62 might have caused this? Alternatively, I'm wondering if you found some workflow to test this in bug 1469287 - there's no indication of the focused item in the DOM/devtools even when it's working properly so it makes it hard to debug.
Comment 7•5 years ago
|
||
(In reply to Brian Grinstead [:bgrins] from comment #6)
Tim, do you have any idea which rule removed in https://hg.mozilla.org/mozilla-central/rev/c8c488989a62 might have caused this?
Very likely: https://hg.mozilla.org/mozilla-central/rev/c8c488989a62#l26.47
Alternatively, I'm wondering if you found some workflow to test this in bug 1469287 - there's no indication of the focused item in the DOM/devtools even when it's working properly so it makes it hard to debug.
I can't remember on top of my head, but I recall this was more easily reproducible on Linux.
Comment 8•5 years ago
|
||
(In reply to Tim Nguyen :ntim from comment #7)
(In reply to Brian Grinstead [:bgrins] from comment #6)
Tim, do you have any idea which rule removed in https://hg.mozilla.org/mozilla-central/rev/c8c488989a62 might have caused this?
Very likely: https://hg.mozilla.org/mozilla-central/rev/c8c488989a62#l26.47
Alternatively, I'm wondering if you found some workflow to test this in bug 1469287 - there's no indication of the focused item in the DOM/devtools even when it's working properly so it makes it hard to debug.
I can't remember on top of my head, but I recall this was more easily reproducible on Linux.
Do you want a patch from us for that? We already have a patch for it that works. It's just getting the same patch as before.
Comment 9•5 years ago
|
||
(In reply to Alex ARNAUD from comment #8)
(In reply to Tim Nguyen :ntim from comment #7)
(In reply to Brian Grinstead [:bgrins] from comment #6)
Tim, do you have any idea which rule removed in https://hg.mozilla.org/mozilla-central/rev/c8c488989a62 might have caused this?
Very likely: https://hg.mozilla.org/mozilla-central/rev/c8c488989a62#l26.47
Alternatively, I'm wondering if you found some workflow to test this in bug 1469287 - there's no indication of the focused item in the DOM/devtools even when it's working properly so it makes it hard to debug.
I can't remember on top of my head, but I recall this was more easily reproducible on Linux.
Do you want a patch from us for that? We already have a patch for it that works. It's just getting the same patch as before.
Feel free to submit one and request review from me or dao.
Assignee | ||
Comment 10•5 years ago
|
||
Updated•5 years ago
|
Assignee | ||
Comment 11•5 years ago
|
||
Assignee | ||
Comment 12•5 years ago
|
||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 13•5 years ago
|
||
Comment 14•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Description
•