On 9/16/2015 7:16 PM, Sergey Bylokhov wrote:
Hi, Shilpi.
Can you clarify, why the test does not work in case of Aqua?
The UI Manager documentation states the following –

“The set of defaults a particular look and feel supports is defined and documented by that look and feel. In addition, each look and feel, or |ComponentUI|provided by a look and feel, may access the defaults at different times in their life cycle. Some look and feels may aggressively look up defaults, so that changing a default may not have an effect after installing the look and feel. Other look and feels may lazily access defaults so that a change to the defaults may effect an existing look and feel. Finally, other look and feels might not configure themselves from the defaults table in any way. None-the-less it is usually the case that a look and feel expects certain defaults, so that in general a |ComponentUI|provided by one look and feel will not work with another look and feel”

If you look at the com.apple.laf.AquaLookAndFeel implementation, it only returns a copy of all the UI defaults (new UIDefaults()) through getDefaults() and any change that you make to the defaults is not going to change the actual defaults. On the other hand, Metal returns you the actual Map that you can modify and customize. As stated in the UI Manager spec, it is up to the look and feel to choose whether to allow customization or not to their defaults and in this case, Aqua has chosen not to. One logical reason I could think of is to retain the parity with native behavior which is the primary intention of any Native Look And Feel.

Regards,
Shilpi

16.09.2015 9:37, shilpi rastogi wrote:
Hi All,

Please review the fix

Bug: https://bugs.openjdk.java.net/browse/JDK-7133530
webrev: http://cr.openjdk.java.net/~psadhukhan/shilpi/webrev/

Issue: Colored text is not shown on disabled checkbox and radio button.
The test expects to be run using Metal L&F but is actually run using Aqua L&F so setting it explicitly.

Thanks,
Shilpi




Reply via email to