RFR: 8321140: Add comment to note difference in Metal's JButton margins [v2]
Alexey Ivanov
aivanov at openjdk.org
Fri Aug 16 16:18:51 UTC 2024
On Thu, 15 Aug 2024 19:12:32 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:
> I moved it to `BasicLookAndFeel.java` since that's where the values are specifically defined.
Perhaps, we need a note in *both files*.
In `MetalLookAndFeel.java` to refer to `BasicLookAndFeel` where the margin is defined, similar to what's done for check boxes and radio button.
And in `BasicLookAndFeel.java` to note that the above margins are vastly different from other L&Fs.
What about this text?
**`MetalLookAndFeel.java`**
// default margin is (2, 14, 2, 14), defined in
// BasicLookAndFeel via "Button.margin" UI property.
**`BasicLookAndFeel.java`**
// The above margin has vastly larger horizontal values when
// compared to other look and feels that don't rely on these values
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20482#discussion_r1720049626
More information about the client-libs-dev
mailing list