675488 - Use the monochrome icons for New Event/Task

Status: RESOLVED FIXED

(Calendar :: Dialogs, defect)

Description

[Richard Marti (:Paenglab)]

Follow the Thunderbird theme under Vista/Win7 and use monochrome icons also in New Event/Task dialog.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: Use monochrome icons

This patch is using !important instead of using every different attribute (:hover, :hover:active, disabled etc.)

Comment 2

[Richard Marti (:Paenglab)]

Attachment: Patch in action

Comment 3

[Richard Marti (:Paenglab)]

Attachment: Use monochrome icons v2

Made a selector more selective to not interfere with TB buttons.

Comment 4

[Andreas Nilsson (:andreasn)]

Comment on attachment 554704 [details] [diff] [review]
Use monochrome icons v2

There seems to be some issues with the customize dialog for that toolbar in that labels don't get icons (except for the save icon).

Comment 5

[Andreas Nilsson (:andreasn)]

Attachment: screenshot ot the issue

Comment 6

[Andreas Nilsson (:andreasn)]

I also wonder if we should set the toolbar to "Icons beside text" instead of "Icons and text", but that can be a separate bug, so I filed bug #687435 about that.

Comment 7

[Decathlon]

(In reply to Andreas Nilsson (:andreasn) from comment #4)
 
> There seems to be some issues with the customize dialog for that toolbar in
> that labels don't get icons (except for the save icon).

It seems a different problem (not caused by the patch). I can see it also in Lightning 1.0b4.
Maybe should another bug be opened?

Comment 8

[Andreas Nilsson (:andreasn)]

(In reply to Decathlon from comment #7)
> (In reply to Andreas Nilsson (:andreasn) from comment #4)
>  
> > There seems to be some issues with the customize dialog for that toolbar in
> > that labels don't get icons (except for the save icon).
> 
> It seems a different problem (not caused by the patch). I can see it also in
> Lightning 1.0b4.
> Maybe should another bug be opened?

Oh, you're right. Changing this to ui-r+ and opening bug #687453 about the missing customize dialog icons.

Comment 9

[Andreas Nilsson (:andreasn)]

Comment on attachment 554704 [details] [diff] [review]
Use monochrome icons v2

Giving ui-r+ since the issue I spotted was not caused by this patch.

Comment 10

[Decathlon]

Comment on attachment 554704 [details] [diff] [review]
Use monochrome icons v2

Apart from my personal hate for the monochrome icons ;-), if we have to necessarily use them, the patch looks good.

A nit here (a semicolon is missing):

>+
>+#event-toolbox > menubar,
>+#event-toolbox > toolbar {
>+  -moz-appearance: none;
>+  border-style: none;
>+  background-color: transparent
>+}

and, just curiosity, why didn't you use directly the id selectors (#event-menubar and #event-toolbar) instead of the child selectors?

r+

Comment 11

[Richard Marti (:Paenglab)]

Attachment: Use monochrome icons v3 for check-in

(In reply to Decathlon from comment #10)
> Comment on attachment 554704 [details] [diff] [review]
> Use monochrome icons v2
> 
> Apart from my personal hate for the monochrome icons ;-), if we have to
> necessarily use them, the patch looks good.
> 
> A nit here (a semicolon is missing):
> 
> >+
> >+#event-toolbox > menubar,
> >+#event-toolbox > toolbar {
> >+  -moz-appearance: none;
> >+  border-style: none;
> >+  background-color: transparent
> >+}
> 
> and, just curiosity, why didn't you use directly the id selectors
> (#event-menubar and #event-toolbar) instead of the child selectors?

Because I must be blind. I changed it in the patch for check-in.

Comment 12

[Richard Marti (:Paenglab)]

Pushed to comm-central

<http://hg.mozilla.org/comm-central/rev/519ee522e7d1>

Comment 13

[Richard Marti (:Paenglab)]

Back ported to comm-aurora:

<http://hg.mozilla.org/releases/comm-aurora/rev/5525d1a7f9b1>

Comment 14

[Richard Marti (:Paenglab)]

Back ported to comm-beta:

<http://hg.mozilla.org/releases/comm-beta/rev/f30c88ade4fa>