> Hi All, > Please review the following fix for jdk17. > > Issue : LookAndFeel.installProperty(list, "opaque", false) is not able to set > the opaque property for JList and JTable. > LookAndFeel.installProperty calls the setUIProperty, and setUIProperty checks > for OPAQUE_SET to change the opaque property as requested by the client. > OPAQUE_SET is always set to true when there is call to setOpaque(boolean).So > when the constructor calls setOpaque(true) OPAQUE_SET is set to true and wont > allow the setUIProperty to change the opaque property. > installProperty should work as the opaque property is not set by the client. > > Fix. Fix is to remove the call to the setOpaque() from the constructor of > JList and JTable. This will allow the client to change the opaque property > calling LookAndFeel.installProperty() when the property is already not set. > > Test : Added a test to check the same. Also tested internal tests which all > are passing. > Link is in JBS.
Tejpal Rebari has updated the pull request incrementally with one additional commit since the last revision: set default opaque property for nimbus JToolTip in SynthToolTipUI instead of skin.laf ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3167/files - new: https://git.openjdk.java.net/jdk/pull/3167/files/953bc3aa..e3b3e8c6 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3167&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3167&range=03-04 Stats: 2 lines in 2 files changed: 1 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/3167.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3167/head:pull/3167 PR: https://git.openjdk.java.net/jdk/pull/3167
