[Dojo-checkins] [dojo] #7333: [regression] Toolbar: disabled buttons not indicated (was: [regression] disabled buttons not indicated)

[dojo]

#7333: [regression] Toolbar: disabled buttons not indicated
--------------------+-------------------------------------------------------
 Reporter:  bill    |        Owner:       
     Type:  defect  |       Status:  new  
 Priority:  normal  |    Milestone:  1.2  
Component:  Dijit   |      Version:  1.1.1
 Severity:  normal  |   Resolution:       
 Keywords:          |  
--------------------+-------------------------------------------------------
Changes (by bill):

  * owner:  liucougar =>
  * component:  Editor => Dijit


Old description:

> In the editor on safari, previously disabled cut/paste buttons showed up
> as gray.  The don't seem to any more.
>
> There seems to be more than one problem (perhaps an issue w/!ToolBar
> itself too?) because on FF3 even in the 1.1 release the button didn't
> show up as disabled.
>
> In trunk, since indent/outdent are disabled except for lists, then should
> show up grayed out.

New description:

 In the editor on safari, previously disabled cut/paste buttons showed up
 as a "washed-out" looking image, less bright/contrasted than the enabled
 state.  The don't seem to any more.

 There seems to be more than one problem (perhaps an issue w/!ToolBar
 itself too?) because on FF3 even in the 1.1 release the button didn't show
 up as disabled.

 In trunk, since indent/outdent are disabled except for lists, then should
 show up grayed out.

--

Comment:

 Due to my change in [13650].    Normal buttons indicate disabled-ness by
 changing the text color from black to gray but that doesn't work for
 toolbar buttons since they don't have text, just an icon.

 I changed the button to not use transparency because it was making the
 button itself transparent and the page background was bleeding through...
 although not sure why we can't just make the image semi-transparent.
 Perhaps because of browser bugs?

-- 
Ticket URL: <http://bugs.dojotoolkit.org/ticket/7333#comment:1>
dojo <http://dojotoolkit.org/>
The Dojo UI Toolkit