create a separate linux theme space for tb3

RESOLVED FIXED in Thunderbird 3.0b2

Status

Thunderbird
General
RESOLVED FIXED
10 years ago
7 years ago

People

(Reporter: monreal, Assigned: Magnus Melin)

Tracking

Trunk
Thunderbird 3.0b2
All
Linux
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

10 years ago
There is much work going on to make Firefox 3 look like a native Linux application at the moment. Widgets are fixed to display using the look of the active GTK theme and icons are either pulled from the desktop theme or redone using the Tango guidelines.

Thunderbird is already getting a few things (fixed widget look) for free, but it would be even better if TB3's theme would also integrate better with matching icons. From what I see, most icons (folders, send, receive, delete...) could just be used from the desktop theme. A few others would need to be shipped in the theme, but luckily most of them have already been done for Firefox 3.
(Reporter)

Comment 1

10 years ago
After having a close look I realized that there are much more icons in TB than what you realize on the first look ;)

So this would not be as straightforward, but there is some interest from some tango artists, so it would certainly be possible.
Add here dependency on bug 415415.

Comment 3

9 years ago
I'm willing to work on this. I use Thunderbird every day, but the tango skin for 2.0 has issues. Would be great if we could give 3.0 the same native look as Firefox 3.

Comment 4

9 years ago
Well, it's been a long time now since this was reported, and I for one would really like to see Thunderbird use native GTK+ icons on Linux.  That's really the only thing keeping me from using Thunderbird.
(Assignee)

Comment 5

9 years ago
FYI, i just landed bug 459286 which takes care of the menu icons part.
Hardware: PC → All

Comment 6

9 years ago
I really think this is a big deal. Using native GTK icons on Linux (with a set of Tango-styled icons for those that are missing) would be a major plus - and we're already doing with Firefox, so why not?
Duplicate of this bug: 349215
(Assignee)

Comment 8

9 years ago
I think I'll just go ahead an hijack this bug for the actual theme space creation, and let bug 415415 handle getting icons for it.
Assignee: nobody → mkmelin+mozilla
Blocks: 415415
Summary: Native look for TB3 default theme → create a separate linux theme space for tb3
(Assignee)

Comment 9

9 years ago
Created attachment 350789 [details] [diff] [review]
proposed fix

Create a gnomestripe theme space, and use it for linux.
Whilst we're doing this, could we drop the intermediate mail/themes/gnomestripe/Makefile.in or combine the two jar.mn files into one in that directory?

This will minimise the file stats that we have to do when building (I need to do a similar change to mail/themes/{qute,pinstripe} sometime, but doing it here for gnomestripe would save a changeset.
(Assignee)

Comment 11

9 years ago
Can't say, my build foo is weak ;)
(In reply to comment #10)
> Whilst we're doing this, could we drop the intermediate
> mail/themes/gnomestripe/Makefile.in or combine the two jar.mn files into one in
> that directory?

(In reply to comment #11)
> Can't say, my build foo is weak ;)

Unless Phil's got any objections, let's go with combining the two jar.mn files into one, so that we therefore only have to open one jar.mn, one output jar etc.
Seems reasonable to me.
(Assignee)

Comment 14

8 years ago
Created attachment 355293 [details] [diff] [review]
proposed fix, v2

Updated patch to work with the fixes from bug 469655.

Still need someone to provide the theme content...
Attachment #350789 - Attachment is obsolete: true
(In reply to comment #14)
> Created an attachment (id=355293) [details]
> proposed fix, v2
> 
> Updated patch to work with the fixes from bug 469655.

A quick glance and this looks fine.

> Still need someone to provide the theme content...

I think that just getting this patch in will enable that to happen, unless you're thinking something should block it. IMHO Even if we don't have anything different straight away, just having the capability to do it will make some of the UI patches easier, especially as I know David had a lot of trouble with getting the new message header working on both Windows & Linux with the same theme.
(Assignee)

Comment 16

8 years ago
Comment on attachment 355293 [details] [diff] [review]
proposed fix, v2

My thinking was it's double work to get fixes in for two theme copies, but maybe that's not so much of a problem considering it's easier for some patches...
Attachment #355293 - Flags: review?(bugzilla)
Attachment #355293 - Flags: review?(bugzilla) → review+
Comment on attachment 355293 [details] [diff] [review]
proposed fix, v2

nit: you need to add the new makefile to mail/makefiles.sh

I'm hoping that having these separated will encourage more work on Linux (or Windows) specifics, and of course make some of our UI patches easier.
(Assignee)

Comment 18

8 years ago
Gnomestripe landed. I'll file a new bug to get rid of some css ifdefs etc we don't need anymore.

changeset:   1590:d693449a3c68
http://hg.mozilla.org/comm-central/rev/d693449a3c68

->FIXED
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
I checked in a bustage fix for this: http://hg.mozilla.org/comm-central/rev/81b5bb466b70
(Assignee)

Updated

8 years ago
Target Milestone: --- → Thunderbird 3.0b2
(Assignee)

Updated

8 years ago
Depends on: 473207
You need to log in before you can comment on or make changes to this bug.