Closed
Bug 2037159
Opened 7 days ago
Closed 4 days ago
Revert "Perform search when users enters text in urlbar and selects search engine"
Categories
(Firefox :: Address Bar, enhancement)
Firefox
Address Bar
Tracking
()
VERIFIED
FIXED
152 Branch
People
(Reporter: daleharvey, Assigned: daleharvey)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [sng][scotchbonnet-followon])
Attachments
(2 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
Accessibility team have voiced concerns over this change in behaviour and recommended reverting it.
Updated•7 days ago
|
| Assignee | ||
Comment 1•7 days ago
|
||
| Assignee | ||
Comment 2•7 days ago
|
||
Pushed by dharvey@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/e6abef4ec437
https://hg.mozilla.org/integration/autoland/rev/ff3809e13d07
Enter search mode when selecting engine from unified search panel. r=search-reviewers,urlbar-reviewers,adw,jteow
Status: NEW → RESOLVED
Closed: 4 days ago
status-firefox152:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
| Assignee | ||
Comment 5•2 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D298668
Updated•2 days ago
|
Attachment #9584889 -
Flags: approval-mozilla-beta?
Comment 6•2 days ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined/Reason for urgency: A change in behaviour was introduced that the accessibility team asked to be reverted. We would really like to avoid exposing the users to the change in behaviour both to avoid being exposed to the regression and also to avoid users learning a new behaviour that changes on the next release
- Code covered by automated testing?: yes
- Fix verified in Nightly?: no
- Needs manual QE testing?: yes
- Steps to reproduce for manual QE testing: Pick an engine from the unified search panel, searchmode should be entered
- Risk associated with taking this patch: low
- Explanation of risk level: Although it is late in the release cycle it is a relatively simple patch
- String changes made/needed?: N/A
- Is Android affected?: no
Flags: qe-verify+
Updated•2 days ago
|
Attachment #9584889 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•2 days ago
|
status-firefox151:
--- → fixed
Updated•1 day ago
|
QA Whiteboard: [uplift][qa-ver-needed-c152/b151]
Comment 8•1 day ago
|
||
Verified the search is no longer performed when choosing a search engine, search mode is opened instead, using Firefox 151 beta 9 and latest Nightly 152.0a1 under Win 11, macOS 15 and Ubuntu 24.
Status: RESOLVED → VERIFIED
QA Whiteboard: [uplift][qa-ver-needed-c152/b151] → [uplift][qa-ver-done-c152/b151]
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•