> The insets for buttons were incorrect for L&Fs except for Aqua when the text 
> is set to HTML. This was fixed in Aqua by adding a conditional to check for 
> the BasicHTML property key in the button component. This same logic can be 
> used to fix Metal & Motif L&Fs in BasicButtonUI, but Nimbus is not fixed by 
> this. Nimbus gets its default values from a skin.laf file, and when the 
> defaults here are set to have left & right insets to 0 for ButtonUI, the 
> issue is fixed. I also tested for non-HTML text after the changes, and the 
> changes do not affect normal text.
> 
> The HtmlButtonImageTest has been changed to cycle through all L&Fs available 
> on a device.

DamonGuy has updated the pull request incrementally with one additional commit 
since the last revision:

  Fixed test summary. Added check for JButton.

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8407/files
  - new: https://git.openjdk.java.net/jdk/pull/8407/files/05bd9a42..cc5091fa

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8407&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8407&range=01-02

  Stats: 4 lines in 2 files changed: 2 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8407.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8407/head:pull/8407

PR: https://git.openjdk.java.net/jdk/pull/8407

Reply via email to