Closed
Bug 1499743
Opened 6 years ago
Closed 6 years ago
Remove the typed restriction character, change the url and search ones
Categories
(Firefox :: Address Bar, enhancement, P1)
Firefox
Address Bar
Tracking
()
VERIFIED
FIXED
Firefox 65
People
(Reporter: mak, Assigned: mak)
References
(Depends on 1 open bug)
Details
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
patch
|
adw
:
review+
jcristau
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
This may need a relnote.
The typed restriction char (~) is not particularly useful due to our bogus typed definition, and we'd like to simplify restriction chars as far as possible.
The url restriction char is now conflicting with the search alias feature, and as such it would be better to change it. We may reuse the ~, or another possibility could be to change it to "|" that is more widely available across layouts, and wouldn't confuse users of ~.
Assignee | ||
Updated•6 years ago
|
Summary: Remove the typed restriction character and change the url one to it → Remove the typed restriction character and evaluate changing the url one to it
Updated•6 years ago
|
Assignee: nobody → adw
Status: NEW → ASSIGNED
Priority: -- → P2
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee: adw → mak77
Assignee | ||
Updated•6 years ago
|
Priority: P2 → P1
Assignee | ||
Comment 1•6 years ago
|
||
Maybe, considered from a while we'd like to change the search restriction char from $ to ? (for things like bug 1315509 and bug 1334019), we could use $ for url restriction. The reason I'd like to avoid ~ and | is because those are more commonly used in command lines and I'd not want to prevent searching those.
There's also an advantage into doing both changes at the same time, where we can just relnote it once, and break habits in a single version, rather than doing muscle memory damage twice (once to change the url restriction and twice to change the search restriction).
Assignee | ||
Updated•6 years ago
|
Summary: Remove the typed restriction character and evaluate changing the url one to it → Remove the typed restriction character, change the url and search ones
Assignee | ||
Comment 2•6 years ago
|
||
Remove the "~" typed restriction character.
Change the url restriction character to "$" and the search one to "?".
Assignee | ||
Comment 3•6 years ago
|
||
For the relnote:
Address Bar restriction characters changed: it is no more possible to restrict results to typed urls ("~"). Matching only on url now uses the "$" character, instead of "@". Restricting results to search suggestions uses the "?" character, instead of "$".
relnote-firefox:
--- → ?
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/4ff8166e2669
Address Bar restriction characters: remove typed, change url and search r=adw
Comment 5•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Assignee | ||
Comment 6•6 years ago
|
||
[Beta/Release Uplift Approval Request]
Feature/Bug causing the regression: None
User impact if declined: We would like to release a blog post regarding the new Search Shortcuts feature in Firefox 64, and it would also point out these additional restriction chars. Changing the chars in 65 would practically made that impossible, so we'd like to change them in 64.
Is this code covered by automated tests?: Yes
Has the fix been verified in Nightly?: Yes
Needs manual test from QE?: Yes
If yes, steps to reproduce: Use restriction characters in the Address Bar
List of other uplifts needed: None
Risk to taking this patch: Low
Why is the change risky/not risky? (and alternatives if risky): It's just changing the chars, it may break muscle memory but it would happen regardless at 65. Needs a rel-note
String changes made/needed:
Attachment #9023777 -
Flags: review?(adw)
Attachment #9023777 -
Flags: approval-mozilla-beta?
Updated•6 years ago
|
Attachment #9023777 -
Flags: review?(adw) → review+
Comment 7•6 years ago
|
||
Comment on attachment 9023777 [details] [diff] [review]
1499743.diff
urlbar shortcuts change to coincide with search shortcuts in 64, approved for 64.0b9
Attachment #9023777 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 8•6 years ago
|
||
bugherder uplift |
status-firefox64:
--- → fixed
Assignee | ||
Comment 9•6 years ago
|
||
We should add this to the nightly and beta release notes.
Beta 64:
Address Bar restriction characters changed: you can match on urls using the "$" character, instead of "@". You can restrict resuts to search suggestions using the "?" character, instead of "$".
Nightly 65:
Address Bar restriction characters changed: it is no more possible to restrict results to typed urls ("~").
Flags: needinfo?(jcristau)
Updated•6 years ago
|
Flags: qe-verify+
Comment 10•6 years ago
|
||
I have reproduced this issue using Firefox 64.0a1(2018.10.17) on Win 10 x64.
I can confirm this issue is fixed, I verified using Firefox 64.0b9 and 65.a1 on Win 10 x64, Ubuntu 18.04 x64 and Mac OS X 10.13.6.
This bug changed 2 of the characters, in particular "$" used to restrict to search suggestions, now we use "?". "@" used to restrict a match to only url (search urls not titles), now we use "$".
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Comment 11•6 years ago
|
||
Added to the draft relnotes, will eventually end up on mozilla.org.
Flags: needinfo?(jcristau)
You need to log in
before you can comment on or make changes to this bug.
Description
•