RFR: 8345004: Backout fix for JDK-8226990
Prasanta Sadhukhan
psadhukhan at openjdk.org
Wed Nov 27 04:33:39 UTC 2024
On Wed, 27 Nov 2024 04:26:18 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> The fix for [JDK-8226990](https://bugs.openjdk.org/browse/JDK-8226990) causes regression in dark theme for GTK L&F on Ubuntu system and that results in manual test failures identified recently.
>>
>> Few of the changes made in [JDK-8226990](https://bugs.openjdk.org/browse/JDK-8226990) is further modified in [JDK-8335130](https://bugs.openjdk.org/browse/JDK-8335130).
>>
>> Backing out the overall fix to prevent any regression.
>>
>> CI testing after revert is fine.
>
> src/java.desktop/share/classes/javax/swing/plaf/synth/SynthTabbedPaneUI.java line 149:
>
>> 147: @Override
>> 148: protected void installDefaults() {
>> 149: updateStyle(tabPane);
>
> guess you missed adding this back
> tabsOpaque = UIManager.getBoolean("TabbedPane.tabsOpaque");
ok..I guess I am wrong..
> src/java.desktop/share/classes/javax/swing/plaf/synth/SynthTabbedPaneUI.java line 640:
>
>> 638: }
>> 639:
>> 640: tabContext.getPainter().paintTabbedPaneTabBackground(tabContext, g,
>
> and this
> if (tabsOpaque || tabPane.isOpaque()) {
it is not needed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22379#discussion_r1859873170
PR Review Comment: https://git.openjdk.org/jdk/pull/22379#discussion_r1859874064
More information about the client-libs-dev
mailing list