Closed
Bug 797266
Opened 12 years ago
Closed 12 years ago
Icons on New Event and New Task buttons don't get the "disabled" look with Aero theme
Categories
(Calendar :: Calendar Frontend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.1
People
(Reporter: bv1578, Assigned: bv1578)
Details
Attachments
(1 file)
|
840 bytes,
patch
|
Paenglab
:
review+
|
Details | Diff | Splinter Review |
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.
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 2•12 years ago
|
||
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
|
|
||
Comment 3•12 years ago
|
||
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/5de93391819b
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1
|
|
||
Updated•12 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•