Closed Bug 1701493 Opened 4 years ago Closed 4 years ago

[Proton] The drop-down background of the select box in the extension inline options page is transparent.

Categories

(Firefox :: Menus, defect, P2)

Firefox 89
Desktop
All
defect

Tracking

()

VERIFIED FIXED
89 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- unaffected
firefox87 --- disabled
firefox88 --- disabled
firefox89 --- verified

People

(Reporter: nayinain, Assigned: Gijs)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [proton-context-menus][priority:2b])

Attachments

(2 files)

Attached video Capture.webm

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

Steps to reproduce:

  1. Set browser.proton.contextmenus.enabled to true in about:config and restart Nightly.
  2. Install the extension (https://addons.mozilla.org/en-US/firefox/addon/traduzir-paginas-web/)
  3. Open the options of the extension in the about:addons page.
  4. lick to open the select box drop-down on the options page.

Actual results:

The drop-down background is transparent.


Sorry for my bad English.

Has STR: --- → yes
Component: Untriaged → Menus
Flags: needinfo?(gijskruitbosch+bugs)

Bug 1692237 broke this.

It looks like the styles at https://searchfox.org/mozilla-central/source/toolkit/themes/windows/global/popup.css#169-176 are applying, but the styles at https://searchfox.org/mozilla-central/rev/d58860eb739af613774c942c3bb61754123e449b/toolkit/themes/windows/global/popup.css#51 are missing the variable definitions, so the CSS variables are not resolving, which is breaking the styling. Although the menupopup itself pulls in global.css which defines these variables, they're defined on :root which is having no effect when loaded into shadow DOM. Also defining the variables on :host fixes this.

Assignee: nobody → gijskruitbosch+bugs
Severity: -- → S2
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(gijskruitbosch+bugs)
OS: Unspecified → All
Regressed by: 1692237
Hardware: Unspecified → Desktop
Whiteboard: [proton-context-menus]
Has Regression Range: --- → yes

Set release status flags based on info from the regressing bug 1692237

Priority: -- → P2
Whiteboard: [proton-context-menus] → [proton-context-menus][priority:2b]
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/72593dfc1e16 make menulist dropdowns in about:addons' add-on options not look completely broken, r=mstriemer
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
Depends on: 1702626
No longer depends on: 1702626
Regressions: 1702626

Reproduced the initial issue using an old Nightly 2021-03-28, verified that this is now fixed using latest Nightly 89.0a1 across platforms (Windows 10, macOS 11.3 and Ubuntu 18.04).

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

Attachment

General

Creator:
Created:
Updated:
Size: