The one-off search buttons are selected too soon when tips are in the results list
Categories
(Firefox :: Address Bar, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox71 | --- | verified |
People
(Reporter: adw, Assigned: bugzilla)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
If the tip is the last result in the popup (e.g. the only result), you key down to select the tip's main button, and then key down again to select the help button, both the help button and the first one-off become selected. I haven't tried to debug it, but it looks like the problem is that we pass in this.view.visibleRowCount
here: https://searchfox.org/mozilla-central/rev/1d0e84d2ec2843924bc6e5ed6658f93439e12351/browser/components/urlbar/UrlbarController.jsm#287
The one-offs use that to determine when the selection moves from the last result to the one-offs. visibleRowCount
is one less than the actual number of selectable elements, so the one-offs think they become selected one selectable element too early.
We should update the tip test for this.
Reporter | ||
Comment 1•5 years ago
|
||
Changing the summary to more accurately reflect the problem.
Reporter | ||
Comment 2•5 years ago
|
||
The one-offs compare view.visibleRowCount
to view.selectedRowIndex
, so not only do we need a view.visibleElementCount
, we also need a view.selectedElementIndex
.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
Comment 5•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Updated•5 years ago
|
Description
•