Closed
Bug 1951649
Opened 10 months ago
Closed 9 months ago
The "Search with" row is expanded when typing a keyword while the opt-in message is displayed
Categories
(Firefox :: Address Bar, defect)
Tracking
()
VERIFIED
FIXED
138 Branch
People
(Reporter: cmuntean, Assigned: daisuke)
References
Details
Attachments
(2 files)
|
860.45 KB,
video/mp4
|
Details | |
|
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
[Affected versions]:
- Firefox Release 136.0 (Build ID: 20250225123537)
- Firefox Beta 137.0b7 (Build ID: 20250303103151)
- Nightly 138.0a1 (Build ID: 20250219215346)
[Affected Platforms]:
- Windows 10 x64
- MacOS 14.7.1
- Linux Mint 20.1
[Prerequisites]:
- Have a Firefox Beta 137 en-US build installed/extracted.
- Have the following prefs set:
- browser.search.region = US
- browser.urlbar.quicksuggest.contextualOptIn = true
[Steps to reproduce]:
- Open the browser with the profile from prerequisites.
- Click in the addressbar.
- While the opt-in message is displayed, type any keyword.
- Observe the "Search with" result.
[Expected result]:
- The "Search with" result has the same dimensions as the other results.
[Actual result]:
- The "Search with" result is extended and has the dimensions of the opt-in message.
[Notes]:
- This issue is also reproducible on Firefox Release 136.0.
- Attached is a screen recording of the issue.
| Assignee | ||
Comment 1•9 months ago
|
||
Updated•9 months ago
|
Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Updated•9 months ago
|
Attachment #9469866 -
Attachment description: Bug 1951649: 'dynamicType' attribute is resetted for each result → Bug 1951649: Remove previously set attributes properly if new result
Pushed by dakatsuka.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/242276177dab
Remove previously set attributes properly if new result r=adw
Comment 3•9 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
Comment 4•9 months ago
|
||
The patch landed in nightly and beta is affected.
:daisuke, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox137towontfix.
For more information, please visit BugBot documentation.
Flags: needinfo?(daisuke)
| Assignee | ||
Comment 5•9 months ago
|
||
Comment on attachment 9469866 [details]
Bug 1951649: Remove previously set attributes properly if new result
Beta/Release Uplift Approval Request
- User impact if declined/Reason for urgency: The style of the results at the top of the page is broken immediately after the results of the context opt-in are displayed.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: Bug 1951651
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The main change is only one line, and affects only urlbar results.
- String changes made/needed: None
- Is Android affected?: Unknown
Flags: needinfo?(daisuke)
Attachment #9469866 -
Flags: approval-mozilla-beta?
| Reporter | ||
Comment 6•9 months ago
|
||
I have verified this issue on the latest Nightly 138.0a1 (Build ID: 20250306213030) on Windows 11 x64, macOS 14.7.1 and Linux Mint 20.1 x64.
- The issue is no longer reproducible, the "Search with" result has the same dimensions as the other results.
I will verify the issue on Beta 137 as soon as it's uplifted.
Status: RESOLVED → VERIFIED
Updated•9 months ago
|
Attachment #9469866 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•9 months ago
|
Updated•9 months ago
|
| Reporter | ||
Comment 8•9 months ago
|
||
I have verified the issue on the latest Beta 137.0b4 (Build ID: 20250310091737) on Windows 10 x64, macOS 14.7.1 and Linux Mint 20.1.
- The issue is no longer reproducible, the "Search with" result has the same dimensions as the other results.
You need to log in
before you can comment on or make changes to this bug.
Description
•