[GTK Themes] Further GTK+ theme integration

201 views
Skip to first unread message

Elliot Glaysher (Chromium)

unread,
Nov 9, 2010, 4:58:08 PM11/9/10
to Chromium-dev
Around the time Ambiance & Radiance were released to the world, the
Chrome GTK+ integration started making progressively poorer decisions
on the frame color due to many of the inputs to our heuristics are
gone. Instead of adding more breakable heuristics, I'm proposing that
we just require theme authors to specify the frame color like they
used to with either the MetaFrames class (general case) or a new
ChromeGtkFrame class (specific to chrome case).

http://code.google.com/p/chromium/wiki/LinuxGtkThemeIntegration

Is there anything I haven't thought of? Any concerns or comments?

-- Elliot

Evan Stade

unread,
Nov 9, 2010, 5:02:38 PM11/9/10
to e...@chromium.org, Chromium-dev
do we need colors for background tab + hovered background tab?

-- Evan Stade



--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
   http://groups.google.com/a/chromium.org/group/chromium-dev

Elliot Glaysher (Chromium)

unread,
Nov 9, 2010, 5:10:13 PM11/9/10
to Evan Stade, Chromium-dev
On Tue, Nov 9, 2010 at 2:02 PM, Evan Stade <est...@chromium.org> wrote:
> do we need colors for background tab + hovered background tab?

Maybe? I don't think so because the background tab color is computed
as a function of the foreground tab color (gtk default) and the
background color (this new color).

-- Elliot

Evan Martin

unread,
Nov 9, 2010, 5:17:00 PM11/9/10
to e...@chromium.org, Chromium-dev
On Tue, Nov 9, 2010 at 1:58 PM, Elliot Glaysher (Chromium)
<e...@chromium.org> wrote:

I wonder if we would be better off not making this as Chrome-specific.
Like, suppose another program cares about matching the window frame color.
In that case, we could call it GtkWindowFrame or something. Not sure
if it's rude to pollute their namespace though.

Evan Stade

unread,
Nov 9, 2010, 5:21:46 PM11/9/10
to Elliot Glaysher (Chromium), Chromium-dev
That seems like a fine fallback, but perhaps providing a setting which we will respect in case they want to override that default is worthwhile. The (not gtk) chrome theme system allows/requires theming that color.

Evan Martin

unread,
Nov 10, 2010, 3:20:00 PM11/10/10
to e...@chromium.org, Chromium-dev
On Tue, Nov 9, 2010 at 1:58 PM, Elliot Glaysher (Chromium)
<e...@chromium.org> wrote:
> Is there anything I haven't thought of? Any concerns or comments?

Ooh ooh another one -- might as well tackle scrollbar color while we're at it?

Elliot Glaysher (Chromium)

unread,
Nov 10, 2010, 6:23:05 PM11/10/10
to Evan Martin, Chromium-dev
As I've been working on this feature, I've noticed that many (most?)
themes in the last year(s) don't change their window color when
inactive and just change the text color. I'm not sure what to do about
that. In my current hacked up version of Ambiance, I just lighten 0.7,
though I'm not sure if that's the best

On Wed, Nov 10, 2010 at 12:20 PM, Evan Martin <ev...@chromium.org> wrote:
> Ooh ooh another one -- might as well tackle scrollbar color while we're at it?

This might be doable. Is there a reason we don't currently just read
whatever the colors were set on GtkScrollbar and do our complicated
workaround instead?

-- Elliot

Evan Martin

unread,
Nov 10, 2010, 6:28:26 PM11/10/10
to Elliot Glaysher (Chromium), Chromium-dev
On Wed, Nov 10, 2010 at 3:23 PM, Elliot Glaysher (Chromium)
<e...@chromium.org> wrote:
> As I've been working on this feature, I've noticed that many (most?)
> themes in the last year(s) don't change their window color when
> inactive and just change the text color. I'm not sure what to do about
> that. In my current hacked up version of Ambiance, I just lighten 0.7,
> though I'm not sure if that's the best

I guess you should bring up a bunch of windows, play around with them
a bit, and then use your judgement as to what looks best. I don't
think it's too harmful if we don't change color when we lose focus --
there are other indicators for focus, like the ring we draw.

> On Wed, Nov 10, 2010 at 12:20 PM, Evan Martin <ev...@chromium.org> wrote:
>> Ooh ooh another one -- might as well tackle scrollbar color while we're at it?
>
> This might be doable. Is there a reason we don't currently just read
> whatever the colors were set on GtkScrollbar and do our complicated
> workaround instead?

I believe it's because the colors that were set are not actually used
by the theme, but I'm not sure.

Elliot Glaysher (Chromium)

unread,
Nov 15, 2010, 9:14:13 PM11/15/10
to Evan Martin, Chromium-dev
On Wed, Nov 10, 2010 at 3:28 PM, Evan Martin <ev...@chromium.org> wrote:
>> This might be doable. Is there a reason we don't currently just read
>> whatever the colors were set on GtkScrollbar and do our complicated
>> workaround instead?
>
> I believe it's because the colors that were set are not actually used
> by the theme, but I'm not sure.

OK. I've update the documentation based on what I committed Saturday.
http://code.google.com/p/chromium/wiki/LinuxGtkThemeIntegration has
been updated and now has a bunch of tables of what the various style
properties can do. I'll send this out tomorrow if no one has any
problems with it.

(P.S.: Adding images for close/min/max buttons looks like it's really
friggin hard! The couple of simple ways don't work!)

-- Elliot

Reply all
Reply to author
Forward
0 new messages