RFR: JDK-7154070: in SwingSet2, switching between LaFs it's easy to lose JTable dividers [v4]

Prasanta Sadhukhan psadhukhan at openjdk.org
Mon Feb 6 10:14:13 UTC 2023


On Mon, 6 Feb 2023 09:21:04 GMT, Jayathirth D V <jdv at openjdk.org> wrote:

>> I am not sure what you meant..
>> uninstallDefaults will be called only once during L&F transition from Nimbus to other L&F and
>>  it will set horizontal and vertical lines only if it was set in other L&F before Nimbus L&F takes effect..
>> If other L&F did not have those set, it will not call setShowHorizontal/VerticalLines()
>
> Will uninstallDefaults() be called for GTK -> other LAF transition?
> I was thinking SynthTableUI is common for all SynthLAF's and if we are transitioning from GTK -> Other LAF's there is no need to call setShowHorizontal/VerticalLines(). Because as mentioned in description only Nimbus LAF will have "showGrid" as false and for GTK we will not be calling `table.setShowGrid(false);`

Yes, for GTK showGrid is true so we can add that condition..
Fix updated...

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

PR: https://git.openjdk.org/jdk/pull/12385



More information about the client-libs-dev mailing list