The default bug view has changed. See this FAQ.

The default tab icon used on Mac doesn't exist

RESOLVED FIXED in Thunderbird 18.0

Status

Thunderbird
Theme
--
minor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: florian, Assigned: andreasn)

Tracking

Trunk
Thunderbird 18.0
x86
Mac OS X
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
With a debug build I see this in my terminal:
Chrome file doesn't exist: [...]/obj-tbird/mozilla/dist/DailyDebug.app/Contents/MacOS/chrome/toolkit/skin/classic/global/tree/item.png

An MXR search shows that this file is referenced from:

http://mxr.mozilla.org/comm-central/source/mail/themes/pinstripe/mail/tabmail.css#48
48 .tabmail-tab {
49   list-style-image: url("chrome://global/skin/tree/item.png");
50 }

and from:
334 .alltabs-item {
335   list-style-image: url("chrome://global/skin/tree/item.png");
336 }

This item.png file doesn't exist.

This seems to cause the alltabs menu item for the inbox and email messages to not have any icon.
While the default icon should be fixed, we should probably also see why alltabs menuitem attempt to use it instead of the icon that these tabs have in the tab bar.

Comment 1

5 years ago
item.png was removed in bug 648668 (looks like it was replaced with chrome://mozapps/skin/places/defaultFavicon.png).

Comment 2

5 years ago
(In reply to Stefan [:stefanh] from comment #1)
> item.png was removed in bug 648668 (looks like it was replaced with
> chrome://mozapps/skin/places/defaultFavicon.png).

Oh, well - I don't think that defaultFavicon.png would be suitable here (just looked at it).
(Reporter)

Comment 3

5 years ago
I was about to file a dup of this bug, mostly because I'm annoyed by the noise in the terminal, but it would be good to have some relevant icons displayed in the all tabs menu popup too. CC'ing Andreas

Comment 4

5 years ago
One cheap solution is to take the icon from 2.0 (http://mxr.mozilla.org/mozilla2.0/source/toolkit/themes/pinstripe/global/tree/item.png) and put it in mail/themes.
(Assignee)

Comment 5

5 years ago
Created attachment 641047 [details] [diff] [review]
patch to fix the issue

I haven't tested this, but I think it should do the trick.
(Reporter)

Comment 6

5 years ago
I've just tested attachment 641047 [details] [diff] [review] and I can confirm that the warning in comment 0 no longer exists.

(In reply to Florian Quèze [:florian] [:flo] from comment #0)

> This item.png file doesn't exist.
> 
> This seems to cause the alltabs menu item for the inbox and email messages
> to not have any icon.

This is fixed by the patch here.

> While the default icon should be fixed, we should probably also see why
> alltabs menuitem attempt to use it instead of the icon that these tabs have
> in the tab bar.

I think we should fix this too, but we can handle it in another bug.

I was about to file a dup again for this today. Could we please get the fix reviewed and landed soon? :) (I almost put an r+ on the attachment after testing it and verifying that it works, but I'm not a peer for mail/themes/)
(Reporter)

Comment 7

5 years ago
(In reply to Florian Quèze [:florian] [:flo] from comment #6)

> > While the default icon should be fixed, we should probably also see why
> > alltabs menuitem attempt to use it instead of the icon that these tabs have
> > in the tab bar.
> 
> I think we should fix this too, but we can handle it in another bug.

This has been fixed in bug 464524 :-).
(Assignee)

Comment 8

5 years ago
Comment on attachment 641047 [details] [diff] [review]
patch to fix the issue

Setting up for review.
Attachment #641047 - Flags: review?(richard.marti)
Comment on attachment 641047 [details] [diff] [review]
patch to fix the issue

This looks good.
Attachment #641047 - Flags: review?(richard.marti) → review+
No checkin-needed ?
(Reporter)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/18d35f49e7fd
Assignee: nobody → nisses.mail
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
You need to log in before you can comment on or make changes to this bug.