Closed Bug 1707839 Opened 3 years ago Closed 3 years ago

Switch to tab option is not visible or cut off when the searched tab has longer strings and browser has certain width

Categories

(Firefox :: Address Bar, defect, P2)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
90 Branch
Tracking Status
firefox88 --- disabled
firefox89 --- verified
firefox90 --- verified

People

(Reporter: atrif, Assigned: mak)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-address-bar] [priority:2b] [proton-uplift])

Attachments

(2 files)

Affected versions

  • 90.0a1(20210421212740)
  • 89.0b4 (20210425185730)

Affected platforms

  • Windows10 x64
  • macOS 11.2.3
  • Ubuntu 18.04

Steps to reproduce

  1. Open Firefox with a new profile and a random Amazon product.
  2. Open a new tab and click on the Tabs option.
  3. Type ama and resize the browser horizontally.

Expected result

  • Switch to tab option is displayed for Amazon.

Actual result

  • Switch to tab option is not displayed on certain browser width.

Regression range

  • I will search for one ASAP if there is one.

Notes

  • Attached a screen recording.
  • S2 because the issue is easily reproducible and it's more visible when higher scaling and resolutions are used.
Has Regression Range: --- → no
Has STR: --- → yes

This is bug 1699325 AFAICT. I'll leave it to you all to dupe it if you'd like so that I don't interfere with your Proton planning.

The problem here is that removing the switch to tab badge in Proton made that bug outcome worse for the user, that now has no way to dtistinguish a switch to tab result if the chiclet is pushed out.

Depends on: 1699325
QA Whiteboard: [qa-regression-triage]

Hello,

This issue is not a regression. Accordingly to bug 1699325#c2, we can also reproduce it on builds way back to 75.0, on Windows 10x64.

Thanks!

Priority: -- → P2
Whiteboard: [proton-address-bar] → [proton-address-bar] [priority:2b]

investigating...

Assignee: nobody → mak
Status: NEW → ASSIGNED
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/4b55610e6d23
Ensure the action text stays visible in Address Bar results. r=adw
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

Comment on attachment 9219150 [details]
Bug 1707839 - Ensure the action text stays visible in Address Bar results. r=adw

Beta/Release Uplift Approval Request

  • User impact if declined: Part of MR1 design. The Switch to tab chiclet should always be visible now that we removed the badge on the favicon.
  • 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: Ensure that the Switch to tab chiclet stays visible even if the page has a long title. See comment 0.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Simple and limited css changes
  • String changes made/needed:
Attachment #9219150 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Whiteboard: [proton-address-bar] [priority:2b] → [proton-address-bar] [priority:2b] [proton-uplift]

Comment on attachment 9219150 [details]
Bug 1707839 - Ensure the action text stays visible in Address Bar results. r=adw

Approved for 89 beta 7, thanks.

Attachment #9219150 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-regression-triage] → [qa-regression-triage] [qa-triaged]

Verified as fixed using Firefox 89 beta 8 and latest Nightly 90.0a1 2021-05-04 across platforms.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-regression-triage] [qa-triaged] → [qa-regression-triage]
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: