Closed Bug 797266 Opened 8 years ago Closed 8 years ago

Icons on New Event and New Task buttons don't get the "disabled" look with Aero theme

Categories

(Calendar :: Calendar Frontend, defect)

Lightning 1.7
All
Windows 7
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bv1578, Assigned: bv1578)

Details

Attachments

(1 file)

It happens for the button "New Task" in task view and for "New Event" in the today pane on Windows with Aero theme.

Steps to reproduce:
set all the calendars in read only mode

the mentioned buttons get disabled and the labels text get the grayed out look, but the icons don't change.
Attached patch patchSplinter Review
The patch fixes.
I've also tried to assign the class "toolbarbutton1 calbar-toolbarbutton-1" (the same of the buttons in the toolbar) instead of assign directly the opacity. It works but the text in New task button gets a wrong position.
Attachment #667335 - Flags: review?(richard.marti)
Comment on attachment 667335 [details] [diff] [review]
patch

Good catch, I haven't know this buttons can have a disabled state.

This looks good. Thanks for this patch.
Attachment #667335 - Flags: review?(richard.marti) → review+
Keywords: checkin-needed
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/5de93391819b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1
You need to log in before you can comment on or make changes to this bug.