Search bar displays cut off search engine at the bottom
Categories
(Firefox :: Search, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox-esr91 | --- | verified |
firefox90 | --- | unaffected |
firefox91 | --- | wontfix |
firefox92 | --- | verified |
firefox93 | --- | verified |
People
(Reporter: atrif, Assigned: daisuke)
References
(Regression)
Details
(Keywords: regression)
Attachments
(4 files)
(deleted),
image/png
|
Details | |
(deleted),
application/x-zip-compressed
|
Details | |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-esr91+
|
Details |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-esr91+
|
Details |
Affected versions
- 92.0a1 (20210727094543)
- 91.0b7 (20210725190237)
Affected platforms
- Windows 10x64
- Windows 7x64
Steps to reproduce
- Unfortunately I don’t have some STR for this issue but I attached a profile on which the issue is reproducible (link to profile).
Expected result
- Address bar is displayed as expected.
Actual result
- Address bar displayed cut-off engine.
Regression range
- Mozregression pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=3e2960113395aeb43cfdb56fc74da7debd6b204b&tochange=a5c85f93707c2790de3cceeda5155bb29d371909
Notes
- Attached a screenshot.
- Cannot reproduce the issue on macOS 10.15 using the attached profile.
Reporter | ||
Updated•3 years ago
|
Comment 1•3 years ago
|
||
Can you take a look?
Assignee | ||
Comment 2•3 years ago
|
||
Thank you very much!
But, I could not reproduce this issue yet...
Alexandru,
I imagined that this profile would have made the search bar visible and changed the engine to DDG, but it seemed to be the same as the default settings. This might be a different profile?? If possible, could you send the profile again?
Thanks!
Daisuke
Reporter | ||
Comment 3•3 years ago
|
||
(In reply to Daisuke Akatsuka (:daisuke) from comment #2)
Thank you very much!
But, I could not reproduce this issue yet...Alexandru,
I imagined that this profile would have made the search bar visible and changed the engine to DDG, but it seemed to be the same as the default settings. This might be a different profile?? If possible, could you send the profile again?Thanks!
Daisuke
Oh sorry for that :(. I have managed to make another one. I hope that this is working as expected. Vaguely the steps where:
- Add Search bar between overflow arrow and Sync icon (while in customize) and then change search engine to DDG.
- Resize Fx with search bar opened until search engines are on two rows.
- Enter and exit Fullscreen while the search bar is opened. This needs some tries and resizing to happen.
But I cannot repro every time. If more investigation is needed please let me know. Thank you!
Assignee | ||
Comment 4•3 years ago
|
||
Thank you very much, Alexandru!
I could reproduce it. I will take a look at this issue.
Assignee | ||
Comment 5•3 years ago
|
||
Assignee | ||
Comment 6•3 years ago
|
||
Depends on D122300
Comment 8•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4047f212ac51
https://hg.mozilla.org/mozilla-central/rev/8a9517eb8c59
Comment 9•3 years ago
|
||
The patch landed in nightly and beta is affected.
:daisuke, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Reporter | ||
Comment 10•3 years ago
|
||
Verified fixed with Firefox 93.0a1 (20210816094534) on Windows 10x64 and attached profile from comment 3. Search engines ar no longer cut when using the profile or when following STR from comment 3.
Assignee | ||
Comment 11•3 years ago
|
||
Comment on attachment 9235708 [details]
Bug 1722507: Get rid of extra margin to calculate the height of container correctly.
Beta/Release Uplift Approval Request
- User impact if declined: Style of search engine buttons will be broken a bit.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Main change is only a style and it is one line.
And also, we got rid of unused code. - String changes made/needed:
Assignee | ||
Updated•3 years ago
|
Comment 12•3 years ago
|
||
Comment on attachment 9235708 [details]
Bug 1722507: Get rid of extra margin to calculate the height of container correctly.
Approved for 92.0b5.
Updated•3 years ago
|
Comment 13•3 years ago
|
||
Comment on attachment 9235708 [details]
Bug 1722507: Get rid of extra margin to calculate the height of container correctly.
And 91.1esr.
Updated•3 years ago
|
Comment 14•3 years ago
|
||
bugherder uplift |
Comment 15•3 years ago
|
||
bugherder uplift |
Reporter | ||
Comment 16•3 years ago
|
||
Verified fixed with Firefox 92.0b5 and Firefox 91.1.1esr (20210817212725) from comment 15 on Windows 10x64 using the attached profile from comment 3.
Description
•