Hi Chris,
yes I can! ;) http://bugzilla.qooxdoo.org/show_bug.cgi?id=3991
And I can supply a patch as well...
It seems that just the Appearance was not handling the 'disabled' state.
I did only check this for the 'modern' theme (which I am using), so the same
might apply to the 'classic' theme.
I will attach the patch to the bug as well.
Cheers
Peter
On 2010-08-13 10:49 Christian Hagendorn wrote:
> Hi Peter,
>
> ups, could you please open a bug report for this issue.
>
> Thanks,
> Chris
>
> Am 13.08.2010 10:11, schrieb Peter Schneider:
>> Hi,
>>
>> in qooxdoo 1.2.x (and trunk) disabling of the MenuBar does not disable the
>> focus indicator.
>>
>> To see what I mean paste the attached code in the playground[1].
>> After disabling the MenuBar by the button, the focus indicator still shows up
>> on the menu buttons.
>>
>> I think this is an (small) issue, cause it just doesn't look good ;)
>>
>> Cheers,
>> Peter
>>
>>
>> [1] I am currently not able to 'tinyurl' it...
>> Neither on http://demo.qooxdoo.org/current/playground
>> nor on http://demo.qooxdoo.org/1.2/playground
Index: Appearance.js
===================================================================
--- Appearance.js (Revision 23022)
+++ Appearance.js (Arbeitskopie)
@@ -2278,7 +2278,7 @@
style : function(states)
{
return {
- decorator : states.pressed || states.hovered ? "selected" : undefined,
+ decorator : (states.pressed || states.hovered) && !states.disabled ?
"selected" : undefined,
textColor : states.pressed || states.hovered ? "text-selected" :
undefined,
padding : [ 3, 8 ]
};
------------------------------------------------------------------------------
This SF.net email is sponsored by
Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
