1172242 - Don't duplicate the communicator directory in TB

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Richard Marti (:Paenglab)]

As a preparation to remove the Aero tree in Windows theme I begin with the communicator directory to unify it.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: deDupCommunicator.patch

Instead of using different smileys, I'm using the ones from OS X, because the aero smileys are too faint and I like the flat and more clear appearance of the OS X smileys more.

Comment 2

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 8616375 [details] [diff] [review]
deDupCommunicator.patch

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

I _think_ this is okay, but I do worry a little about the SeaMonkey folks…
So, r=me with the question below answered, and someone who knows how Suite is build saying it's good.  :)

::: mail/themes/windows/jar.mn
@@ -17,5 @@
>  # ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION!
>  # ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION!
>  
> -% skin communicator classic/1.0 %skin/classic/communicator/ os=WINNT osversion<6
> -% skin communicator classic/1.0 %skin/classic/communicator/ os!=WINNT

Don't we still need to specify something for os!=WINNT?
Could we just have:
  % skin communicator classic/1.0 %skin/classic/communicator/
?

Comment 3

[Richard Marti (:Paenglab)]

Attachment: Patch for check-in

(In reply to Blake Winton (:bwinton) from comment #2)
> Comment on attachment 8616375 [details] [diff] [review]
> deDupCommunicator.patch
> 
> Review of attachment 8616375 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I _think_ this is okay, but I do worry a little about the SeaMonkey folks…
> So, r=me with the question below answered, and someone who knows how Suite
> is build saying it's good.  :)
> 
> ::: mail/themes/windows/jar.mn
> @@ -17,5 @@
> >  # ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION!
> >  # ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION! ATTENTION!
> >  
> > -% skin communicator classic/1.0 %skin/classic/communicator/ os=WINNT osversion<6
> > -% skin communicator classic/1.0 %skin/classic/communicator/ os!=WINNT
> 
> Don't we still need to specify something for os!=WINNT?
> Could we just have:
>   % skin communicator classic/1.0 %skin/classic/communicator/
> ?

Yes, I removed the os=WINNT because it's no more needed.

Comment 4

[Magnus Melin [:mkmelin]]

https://hg.mozilla.org/comm-central/rev/850e8b23e028 -> FIXED