Closed Bug 1922802 Opened 10 months ago Closed 9 months ago

Vertical tabs: address bar dropdown is transparent, bookmark toolbar can be clicked through it

Categories

(Firefox :: Address Bar, defect, P1)

Firefox 133
defect

Tracking

()

VERIFIED FIXED
133 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr128 --- unaffected
firefox131 --- unaffected
firefox132 --- unaffected
firefox133 + verified

People

(Reporter: community, Assigned: nsharpley)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [fidefe-sidebar])

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:133.0) Gecko/20100101 Firefox/133.0

Steps to reproduce:

Updated to latest nightly from the website.

Actual results:

Please refer to this (closed) bug as well: https://bugzilla.mozilla.org/show_bug.cgi?id=1922546#c12

This has been marked as fixed, but as far as I can see the z-index of the bookmarks toolbar is now the same as the z-index of the dropdown menu. This is with vertical tabs enabled.

Expected results:

z-index of the dropdown should be greater than the z-index for the bookmarks toolbar.

The Bugbug bot thinks this bug should belong to the 'Firefox::Bookmarks & History' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Bookmarks & History
Component: Bookmarks & History → Address Bar
Duplicate of this bug: 1922827
Summary: Address bar dropdown has same z-index as the bookmarks toolbar → Vertical tabs: address bar dropdown is transparent, bookmarks can be clicked through it
Summary: Vertical tabs: address bar dropdown is transparent, bookmarks can be clicked through it → Vertical tabs: address bar dropdown is transparent, bookmark toolbar can be clicked through it

I ran a mozregression for Bug 1922827 and the bottom of the regression showed:

Bug 1899598 - Hide the horizontal tab strip when vertical tabs are enabled. r=mconley,desktop-theme-reviewers,tabbrowser-reviewers,sidebar-reviewers,emilio,willdurand,sclements

  • Remove the #titlebar element, and ensure toolbars that comprise the titlebar get a .browser-titlebar class
  • Ensure we call TabBarVisibility.update when initializing with verticalTab=true
  • Adjust TabBarVisibility logic so we allow for the vertical tabs case
  • Give #navigator-toolbar z-index: var(--browser-area-z-index-toolbox) to ensure the urlbar view doesnt get clipped behind the lower elements

Differential Revision: https://phabricator.services.mozilla.com/D222059

2024-10-04T11:17:35.201000: DEBUG : Did not find a branch, checking all integration branches
2024-10-04T11:17:35.206000: INFO : The bisection is done.
2024-10-04T11:17:35.209000: INFO : Stopped

[Tracking Requested - why for this release]: address bar not functional with optional vertical tabs enabled.

Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(sfoster)
Keywords: regression
Regressed by: 1899598
Regressions: 1922898
No longer regressions: 1922898
Whiteboard: [fidefe-sidebar]

The bug is marked as tracked for firefox133 (nightly). However, the bug still isn't assigned.

:cbellini, could you please find an assignee for this tracked bug? Given that it is a regression and we know the cause, we could also simply backout the regressor. If you disagree with the tracking decision, please talk with the release managers.

For more information, please visit BugBot documentation.

Flags: needinfo?(cbellini)
Duplicate of this bug: 1923104
Duplicate of this bug: 1923127
Assignee: nobody → nsharpley
Status: NEW → ASSIGNED
Severity: -- → S2
Priority: -- → P1
See Also: → 1922546

:nikkis is looking at this one.

Flags: needinfo?(sfoster)
Flags: needinfo?(cbellini)
Attachment #9429377 - Attachment description: Bug 1922802 - Set z-index on nav bar in vertical tabs mode to prevent overlapping address dropdown r=#sidebar-reviewers → Bug 1922802 - Set z-index on nav bar to prevent PersonalToolbar overlapping address dropdown r=#sidebar-reviewers
Duplicate of this bug: 1923451
Duplicate of this bug: 1923629
Duplicate of this bug: 1923611

This bug just appeared with the latest update. I had nothing at all before.

Pushed by nsharpley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ba57d03292b6 Set z-index on nav bar to prevent PersonalToolbar overlapping address dropdown r=sidebar-reviewers,desktop-theme-reviewers,sfoster
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 133 Branch

Updated today, so far everything is working fine. (Nightly Version 133.0a1 build ID 20241009214609).

Regressions: 1923836

Backed out as requested for causing Bug 1923836.

Status: RESOLVED → REOPENED
Flags: needinfo?(nsharpley)
Resolution: FIXED → ---
Target Milestone: 133 Branch → ---
Backout by sstanca@mozilla.com: https://hg.mozilla.org/mozilla-central/rev/c1e76f8db405 Backed out changeset ba57d03292b6 as requested for causing Bug 1923836.
Depends on: 1921811
Duplicate of this bug: 1924367
Duplicate of this bug: 1924380
Attachment #9429377 - Attachment is obsolete: true
Duplicate of this bug: 1925960

This looks to have been fixed by the fix for bug 1921811 testing on windows 11 and mac with nightly 2024-10-22

Status: REOPENED → RESOLVED
Closed: 10 months ago9 months ago
Flags: needinfo?(nsharpley)
Resolution: --- → FIXED
Duplicate of this bug: 1925278
Target Milestone: --- → 133 Branch
Duplicate of this bug: 1926217
Flags: qe-verify+

I was able to reproduce the issue on an affected Firefox Nightly build from 2024-10-09, using Windows 11, as shown in Comment 0.
Verified as fixed on Firefox 133.0b8 using Windows 11, macOS 14.7 and Ubuntu 22.04. The address bar dropdown is no longer transparent.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: