Bug 235706 – [GTK] Support accent colors

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

[Alice Mikhaylenko]

See the patch. Had it lying around for a while, probably forgot to submit.

Untested on WPE, let's see what EWS says. There is logic to fall back to the previous blue if no accent was set, but I don't know if it builds.

[Alice Mikhaylenko]

Created attachment 450127 [details]
Patch

[EWS Watchlist]

Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

[Michael Catanzaro]

Comment on attachment 450127 [details]
Patch

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

> Source/WebCore/ChangeLog:9
> +        here. Remove the last bits of ThemeGtk and foreign drawing on GTK3.

You haven't really removed the last bits, since we still draw GTK-style scrollbars.

> Source/WebKit/UIProcess/API/gtk/WebKitWebViewBase.cpp:1847
> +#if USE(GTK4)
> +static void webkitWebViewBaseCssChanged(GtkWidget* widget, GtkCssStyleChange* change)
> +#else
> +static void webkitWebViewBaseStyleUpdated(GtkWidget* widget)
> +#endif
> +{
> +#if USE(GTK4)
> +    GTK_WIDGET_CLASS(webkit_web_view_base_parent_class)->css_changed(widget, change);
> +#else
> +    GTK_WIDGET_CLASS(webkit_web_view_base_parent_class)->style_updated(widget);
> +#endif
> +
> +    WebKitWebViewBase* webViewBase = WEBKIT_WEB_VIEW_BASE(widget);
> +    WebKitWebViewBasePrivate* priv = webViewBase->priv;
> +    priv->pageProxy->accentColorDidChange();
> +}

Style nit: it's more readable to duplicate this short function for GTK 4 vs. GTK 3 than to have all the #ifs inside of it.

[Alice Mikhaylenko]

Created attachment 450138 [details]
Patch

[EWS]

Committed r288679 (246485@main): <https://commits.webkit.org/246485@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 450138 [details].