Improve one-off search button layout in very narrow windows
Categories
(Firefox :: Address Bar, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox-esr68 | --- | unaffected |
firefox69 | --- | unaffected |
firefox70 | --- | verified |
firefox71 | --- | verified |
People
(Reporter: cfogel, Assigned: dao)
References
(Regression)
Details
(Keywords: regression)
Attachments
(3 files, 1 obsolete file)
35.39 KB,
image/png
|
Details | |
47 bytes,
text/x-phabricator-request
|
lizzard
:
approval-mozilla-beta+
|
Details | Review |
38.95 KB,
image/png
|
Details |
Affected versions
- 70.0b7, 71.0a1 (2019-09-05) ;
Affected platforms
- Windows 10, Ubuntu 18.40, macOS 10.12;
Steps to reproduce
- Launch Firefox ;
- Resize the browser to @ 1/4 width of the screen;
- Click to open the address bar;
Expected result
- One-of search options are displayed in the screen;
Actual result
- The dropdown has a min-width that is fixed and does not take in account the resize;
- buttons get truncated instead of being shifted into a visible position(space is available for it);
Regression range
- Potential regressor: 1561894
Additional notes
- attached screenshot with the issue;
Reporter | ||
Comment 1•5 years ago
|
||
Adding Harry for a second opinion; in case this issue can be fixed with another existing bug.
Assignee | ||
Comment 2•5 years ago
|
||
(In reply to Cristian Fogel, QA [:cfogel] from comment #0)
Affected versions
- 70.0b7, 71.0a1 (2019-09-05) ;
This appears to be an old Nightly, please update.
Reporter | ||
Comment 3•5 years ago
|
||
Good point.
However, while with 71.0a1 (2019-09-16) the buttons are somewhat in order but the text before them is still fixed and taking up allot of the available space.
Comment 4•5 years ago
|
||
This is working as intended in comment 3's screenshot. I'm cc'ing Verdi just so he's aware of the UX here.
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Verdi produced a spec for this that resizes with the page: https://firefox.michaelverdi.com/squish/. The minimum row width should match the length of the localized string.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 7•5 years ago
|
||
I'm going to post a simple upliftable patch that I think addresses the main concern here and file a followup for verdi's more complex layout proposal.
Assignee | ||
Comment 8•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment 9•5 years ago
|
||
Please remember to file the follow-up today, and make it block the megabar, we must meetup to discuss the path forward.
Comment 10•5 years ago
|
||
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c6ceaf18bea3 Improve one-off search button layout in narrow windows. r=mak
Comment 11•5 years ago
|
||
bugherder |
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 12•5 years ago
|
||
Comment on attachment 9095544 [details]
Bug 1581753 - Improve one-off search button layout in narrow windows. r=mak
Beta/Release Uplift Approval Request
- User impact if declined: see comment 3
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: reduce the window width and type something 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): Pretty simple fix
- String changes made/needed:
Reporter | ||
Comment 13•5 years ago
|
||
With 71.0a1(2019-09-29) the main issue seems to be addressed.
However as per comment 6, one more improvement can be made for this case; the buttons being pushed bellow the "This time, search with:" text when bellow a certain width.
Should we file a separate bug/enhancement for it?
Assignee | ||
Comment 14•5 years ago
|
||
(In reply to Cristian Fogel, QA [:cfogel] from comment #13)
With 71.0a1(2019-09-29) the main issue seems to be addressed.
However as per comment 6, one more improvement can be made for this case; the buttons being pushed bellow the "This time, search with:" text when bellow a certain width.Should we file a separate bug/enhancement for it?
We already filed bug 1584253 on this.
Reporter | ||
Comment 15•5 years ago
|
||
Awesome, thank you for the prompt reply!
Marking the issue verified for nightly.
Keeping the QE flag until we get the beta-fix approved & checked.
Comment 16•5 years ago
|
||
Comment on attachment 9095544 [details]
Bug 1581753 - Improve one-off search button layout in narrow windows. r=mak
CSS fix, verified in nightly, let's uplift for beta 12.
Comment 17•5 years ago
|
||
bugherder uplift |
Updated•5 years ago
|
Description
•