Bug 48099 – [GTK] Menu list buttons do not react properly to CSS styling

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Martin Robinson]

There are a couple issues with menu list buttons. The first is that they ignore many important CSS properties like width, height, and padding. In particular, the GTK+ values override any padding set on the widget.

The second issue is that when certain CSS properties are set, WebKit falls back rendering via calls to adjustMenuListButtonStyle and paintMenuListButton (versus adjustMenuListStyle and paintMenuList). This distinction is for the benefit of the Mac port, which tries to render Aqua-themed controls whenever possible.

At the current time, we always render GTK+-themed controls, so we can ignore the split and implement both similarly.

[Martin Robinson]

Created attachment 71516 [details]
Patch for this issue

[Xan Lopez]

Comment on attachment 71516 [details]
Patch for this issue

View in context: https://bugs.webkit.org/attachment.cgi?id=71516&action=review

Seems sensible.

> WebCore/ChangeLog:5
> +        [GTK] Menu list buttons do react properly to CSS styling

Is there a 'not' missing here or am I missing something?

[Martin Robinson]

Comment on attachment 71516 [details]
Patch for this issue

Clearing flags on attachment: 71516

Committed r70391: <http://trac.webkit.org/changeset/70391>

[Martin Robinson]

All reviewed patches have been landed.  Closing bug.

[WebKit Review Bot]

http://trac.webkit.org/changeset/70391 might have broken GTK Linux 64-bit Debug
The following tests are not passing:
transforms/2d/zoom-menulist.html