Closed Bug 406742 Opened 13 years ago Closed 11 years ago

create a separate linux theme space for tb3

Categories

(Thunderbird :: General, defect)

All
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b2

People

(Reporter: micmon, Assigned: mkmelin)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
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.
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.
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.
FYI, i just landed bug 459286 which takes care of the menu icons part.
Hardware: PC → All
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
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
Attached patch proposed fix (obsolete) — Splinter Review
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.
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.
Attached patch proposed fix, v2Splinter Review
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.
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.
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
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b2
Depends on: 473207
You need to log in before you can comment on or make changes to this bug.