Closed Bug 1708136 Opened 4 years ago Closed 3 years ago

Autocomplete panels don't get shadows on WebRender / Wayland on Linux

Categories

(Toolkit :: Themes, defect)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1738084

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(2 obsolete files)

Spun off bug 1706154, see the discussion there. It seems GTK only draws shadows on known opaque surfaces, and only on X11, so both WebRender (which always uses alpha visuals) and Wayland (which never decorates the popup) don't show shadows.

Use the same setup as arrow panels.

Assignee: nobody → emilio
Status: NEW → ASSIGNED
Component: Graphics: Layers → Themes
Product: Core → Toolkit
Summary: Autocomplete panels don't get shadows on WebRender / Wayland → Autocomplete panels don't get shadows on WebRender / Wayland on Linux

Dao, is something like this (I've only tested on Linux, needs work on Windows / macOS) what you had in mind? If so I can probably get it green etc...

https://hg.mozilla.org/try/rev/1c0432ffb21dd9f5db313728c71352e9e4fbb61d

Flags: needinfo?(dao+bmo)

Yeah, that looks promising.

Flags: needinfo?(dao+bmo)
See Also: → 1706378
Depends on: 1708735

Share the concept of a panel content with all other menupopups / panels.

This avoids importing global.css in the shadow tree, and renames the
arrowcontent part to just "content", since we want to introduce a
"content" part for other panels.

This shouldn't change behavior but makes bug 1708136 a matter of
tweaking a couple CSS rules and fixing up test failures.

Comment on attachment 9219563 [details]
Bug 1708136 - Make the arrowpanel code more generic. r=dao

Revision D113990 was moved to bug 1708735. Setting attachment 9219563 [details] to obsolete.

Attachment #9219563 - Attachment is obsolete: true
See Also: → 1507817
Attachment #9218895 - Attachment is obsolete: true

This should work now.

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME

(Duping to the right bug)

Resolution: WORKSFORME → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: