[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] FYI: MetalLookAndFeel
From: |
Roman Kennke |
Subject: |
Re: [cp-patches] FYI: MetalLookAndFeel |
Date: |
Tue, 20 Sep 2005 14:21:51 +0200 |
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.
BTW, I could not figure out where this method is actually used. The text
components surely have a different border (looks like they inherit their
borders from the BasicLookAndFeel, except for the text fields).
/Roman
signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil