Convert typed bookmarks, tabs, and history restriction characters/tokens to search mode
Categories
(Firefox :: Address Bar, task, P2)
Tracking
()
People
(Reporter: yoasif, Assigned: adw)
References
Details
(Keywords: nightly-community)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Steps to reproduce:
- type %
- type a query
What happens:
The "tabs" icon/scope is not indicated; doesn't have an affordance.
Expected result:
I should have something in the UI that shows what search is selected in the scope bar.
An example of this can be seen in https://developer.apple.com/design/human-interface-guidelines/macos/buttons/scope-buttons/
Reporter | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
I thought we had a bug for converting typed restriction chars, like %
, to search mode (with the visible search mode indicator), but it doesn't look like it. That's the plan though, at least for the three local one-offs we have right now (bookmarks, tabs, history). This bug can be about that.
Assignee | ||
Updated•4 years ago
|
Reporter | ||
Comment 2•4 years ago
|
||
Drew, that is also an issue, but the issue I was noting was about the affordance in the scope bar, not the address bar itself. Should I file a new bug? It looks exactly the same as this one for non-local searches (as they already show an affordance for the restriction character in the address bar).
Reporter | ||
Comment 4•4 years ago
|
||
The search engine and local icons in the "this time search with" row.
Assignee | ||
Comment 5•4 years ago
|
||
The scope will be shown in the blue search mode indicator in the urlbar. UX hasn't indicated that we should also highlight/select the one-off, but we can ask them about that.
Reporter | ||
Comment 6•4 years ago
|
||
Given that the scope can be activated via the scope bar (and the keyword/restriction characters seem to be a shortcut), I think this makes the connection via scope and suggestions more obvious. This is widely used (see apple HIG link above) and is an affordance beyond the keyword (which is frankly not all that discoverable).
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 7•4 years ago
|
||
This fixes this bug (bug 1658964), bug 1659714, bug 1659738, and bug 1659032.
They're all kind of related. It still needs more tests.
Reporter | ||
Updated•4 years ago
|
Assignee | ||
Comment 8•4 years ago
|
||
Assignee | ||
Comment 9•4 years ago
|
||
Assignee | ||
Comment 10•4 years ago
|
||
With update2 enabled (as usual on Nightly): https://treeherder.mozilla.org/#/jobs?repo=try&revision=36b2d8f376a835567dd52420f539e3a4a955bd98
With update2 disabled to simulate non-Nightly: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d970ba341e64e6189744846c2f35aa2462f19b92
Assignee | ||
Comment 11•4 years ago
|
||
Try pushes with the latest m-c plus several urlbar patches on autoland, with a small test failure in https://phabricator.services.mozilla.com/D87510 also fixed:
With update2 enabled: https://treeherder.mozilla.org/#/jobs?repo=try&revision=08d8605658780b1a8adda56991e99648dce2a1d7
Disabled: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b85f653f0b2ba844a29bef04f3767d4fdec2789
Assignee | ||
Comment 12•4 years ago
|
||
Updated•4 years ago
|
Comment 13•4 years ago
|
||
Comment 14•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Description
•