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

RESOLVED FIXED in 2.1

Status

Calendar
Calendar Views
--
trivial
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Decathlon, Assigned: Decathlon)

Tracking

Lightning 1.7
All
Windows 7

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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.
(Assignee)

Comment 1

5 years ago
Created attachment 667335 [details] [diff] [review]
patch

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+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/5de93391819b
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.