257456 - Hook up small-icon, icons-only and text-only mode for Sunbird's customizable toolbars feature

Status: VERIFIED FIXED

(Calendar :: Sunbird Only, defect)

Description

[Simon Paquet [:sipaq]]

Patch coming...

Comment 1

[Simon Paquet [:sipaq]]

Attachment: patch v1(checked in)

Comment 2

[Mostafa Hosseini]

Comment on attachment 157436 [details] [diff] [review]
patch v1(checked in)


Although this bug depends on bug 250867, we won't wait for it to be fixed but
once it is, we can back out these changes and use global css

Comment 3

[Simon Paquet [:sipaq]]

Attachment: Hook up small icons

Comment 4

[Simon Paquet [:sipaq]]

Attachment: Hook up small icons - v2

I found some small mistakes in my patch.

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

Please create patches from the mozilla dir, not way deep into the directory
tree. It makes it hard to apply them. (you have to read them to find out where
to manually cd to etc..)

Comment 6

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 157470 [details] [diff] [review]
Hook up small icons - v2

>+toolbar[iconsize="small"] #calendar-new-event-button:hover:active {
>+}
Those empty rules are a kind of useless, but since the large buttons have them
to, let it be. We can remove the rules in a seperate bug.

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

Checked in small-icons patch.

Comment 8

[Simon Paquet [:sipaq]]

(In reply to comment #6)

> >+toolbar[iconsize="small"] #calendar-new-event-button:hover:active {
> >+}
> Those empty rules are a kind of useless, but since the large buttons have them
> to, let it be. We can remove the rules in a seperate bug.

I filed bug 319426 about it.

Comment 9

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: Sunbird Only. Be afraid for your sanity!