Closed Bug 1520449 Opened 6 years ago Closed 6 years ago

Do the search hand-off on keydown instead of mouse click

Categories

(Firefox :: New Tab Page, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 66
Iteration:
66.3 - Jan 7 - 20
Tracking Status
firefox66 --- verified

People

(Reporter: rrosario, Assigned: rrosario)

References

Details

(Keywords: github-merged)

Attachments

(1 file)

The current search handoff implementation switches focus to the awesomebar when the user mouse clicks on the in-content search bar (but the styles are faked so the user doesn't notice). This turned out to be unnecessary. In the case of keyboard navigation, the focus isn't switched until the user types a printable character. This is working fine so we should switch the mouse navigation case to do the same.

Attached file GitHub Pull Request (deleted) —

There is already an approved pull request for the change. Just waiting for :k88hudson to give the final nod.

Flags: needinfo?(khudson)

If we make this change, we can undo the changes from bug 1514352 and wontfix bug 1515044.

Commit pushed to master at https://github.com/mozilla/activity-stream https://github.com/mozilla/activity-stream/commit/b86a6e91949e43781c19714215af747a13cb104d Bug 1520449 - Do the search hand-off on keydown instead of mouse click (#4650)
Keywords: github-merged

Notes for QA:

Please verify the following cases are working properly:

  • tab to the in-content search and start typing
  • tab to the in-content search and paste some content
  • tab to the in-content search and hit <Enter> (should shift focus to awesome bar)
  • mouse click on the in-content search and start typing
  • right-click on the in-content search and paste some content via context menu

Also, it would be great if this could be tested on a slower machine to make sure there are no issues with lag.

Flags: needinfo?(khudson)
Blocks: 1520691
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Depends on: 1521446

I have verified that the issue is no longer reproducible on the latest Nightly 66.0a1 (Build ID 20190120213632) on Windows 10 x64, Mac OS 10.13.3, and Arch Linux 4.14. I've verified all of the scenarios from comment 4.

I have observed a bit of lag when you make a 2nd search that generates suggestions, where the suggestions from the 1st search will briefly appear before the new ones take their place. But this is also reproducible with the awesome bar alone.

Status: RESOLVED → VERIFIED
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: