Re: [cp-patches] FYI: MetalLookAndFeel

[Roman Kennke]

Hi,

> Am Dienstag, den 20.09.2005, 12:02 +0000 schrieb David Gilbert:
>> Roman Kennke wrote:
>>
>> >I added some defaults to the MetalLookAndFeel (text field borders).
>> >
>> >-      "TextField.border", MetalBorders.getTextFieldBorder(),
>> >+      "TextField.border",
>> >+      new BorderUIResource(MetalBorders.getTextFieldBorder()),
>> >
>> >
>> >
>> Hi Roman,
>>
>> When I saw this I wondered if the getTextFieldBorder() method should
>> return a border that already implements UIResource.  I added some Mauve
>> tests that confirm that it does, so I think this change should be
>> reverted and getTextFieldBorder() fixed instead.
>
> Yeah ok, I'll do this and check it in. What about the getTextBorder()
> method? I assume it'll be the same here.

Ok, I have fixed this.

2005-09-20  Roman Kennke  <address@hidden>

        * javax/swing/plaf/metal/MetalBorders.java
        (getTextFieldBorder): This method must return an BorderUIResource
        instance. Fixed.

/Roman

MetalBorders.diff
Description: Text Data