848292 - Icons for special IMAP folder (Inbox, trash, ...) are not shown when folder is shared

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Tony Mechelynck [:tonymec]]

+++ This bug was initially created as a clone of Bug #840003 +++

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:18.0) Gecko/20100101 Firefox/18.0 SeaMonkey/2.15.2
Build ID: 20130203000651

Steps to reproduce:

Setup imap account.


Actual results:

Special folder (Inbox trash, Junk, ...)  have the generic shared folder icon when these folders are shared (they can't be unshared). See attachement.


Expected results:

Special folders should have special icons.

Comment 1

[Tony Mechelynck [:tonymec]]

Attachment: patch v0 (rules commented-out)

On closer inspection of the source, Thunderbird should not suffer from the bug because the shared folder rules are commented-out. They are at the wrong place though.

This first version of the patch moves them at a better position (earlier in the bug, i.e., at a lesser priority) and adds a placeholder for gnomestripe which did not even have a commented-out rule.

Comment 2

[Tony Mechelynck [:tonymec]]

removing dependency because at this stage both bugs can evolve in parallel

Comment 3

[Richard Marti (:Paenglab)]

Comment on attachment 721658 [details] [diff] [review]
patch v0 (rules commented-out)

Review of attachment 721658 [details] [diff] [review]:
-----------------------------------------------------------------

Thank you Tony for the patch. If we use it in the future then it's correctly before the new mail rule.

r+ with the comment addressed.

::: mail/themes/gnomestripe/mail/folderPane.css
@@ +22,5 @@
> +/* ..... Shared folders ..... */
> +treechildren::-moz-tree-image(folderNameCol, imapShared-true) {
> +  /* no rule yet in this theme */
> +}
> +

Please can you comment out the whole rule like in the other themes? You can also remove the line /* no rule yet in this theme */ and let it epty.

Comment 4

[Tony Mechelynck [:tonymec]]

Attachment: patch v0.1 (r+=Paenglab) addressing comment #3

Comment 5

[Tony Mechelynck [:tonymec]]

I haven't checked that the image-part referenced by coordinates in the commented-out rule for qute is the right one, or that the separate "folder-share.png" for pinstripe exists. If they aren't as should be, or if different icons are desired for shared folders, then this bug ought to block the Thunderbird equivalent (if there is one, or when there will be one) of SeaMonkey bug 132927.

Similarly if any "shared" icon at all is desired for gnomestripe.

Oh, and BTW, please excuse my naivety: if pinstripe is for Mac, and gnomestripe and qute are for Linux GTK2 and Qt respectively, where is the theme for Windows?

Comment 6

[Tony Mechelynck [:tonymec]]

I've just posted a message on mozilla.dev.apps.thunderbird (and mozilla.dev.apps.seamonkey for bug 840003) in the hope that the authors of existing third-party themes will see it, and, if necessary, take the required action.

Comment 7

[[e-mail]]

(In reply to Tony Mechelynck from comment #5)
> Oh, and BTW, please excuse my naivety: if pinstripe is for Mac, and
> gnomestripe and qute are for Linux GTK2 and Qt respectively, where is the
> theme for Windows?

Both GTK2 and Qt use gnomestripe; Windows uses qute. (Although I guess they should be renamed in light of the toolkit renaming...)

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/68a3857fbcab