<Swing Dev> [9] Review request for 8152159 LabelUI is not updated for TitledBorder

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Fri Mar 18 18:29:18 UTC 2016


On 18/03/16 19:40, Semyon Sadetsky wrote:
> Hi Alexander,
>
> Maybe it would be more optimal to update UI in a property change 
> listener added to the UIManager?
>
>     public void propertyChange(PropertyChangeEvent e) {
>         if ("lookAndFeel" == e.getPropertyName()) {
>               label.updateUI();
>         }
>     }
>

   Thank you for the review.

   Could you review the updated fix:
     http://cr.openjdk.java.net/~alexsch/8152159/webrev.01/

   - property change listener is used to get a notification bot L&F or 
LabelUI change
   - the test is updated to check the LabelUI change

  Thanks,
  Alexandr.
> --Semyon
>
> On 3/18/2016 11:58 AM, Alexander Scherbatiy wrote:
>>
>> Hello,
>>
>> Could you review the fix:
>>   bug: https://bugs.openjdk.java.net/browse/JDK-8152159
>>   webrev: http://cr.openjdk.java.net/~alexsch/8152159/webrev.00
>>
>>
>>   The TitledBorder label is only used for painting and does not 
>> belong  to any component hierarchy so it is not updated by 
>> SwingUtilities.updateComponentTreeUI() method.
>>   The fix updates the  TitledBorder label UI when the label is 
>> requested.
>>
>>   Thanks,
>>   Alexandr.
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160318/58013677/attachment.html>


More information about the swing-dev mailing list