8000 Properly reset the EmbeddedSwingComposite when disposed by ptziegler · Pull Request #1231 · eclipse-windowbuilder/windowbuilder · GitHub
[go: up one dir, main page]

Skip to content

Properly reset the EmbeddedSwingComposite when disposed#1231

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:swing-reset-embedded-swing-composite
Sep 17, 2025
Merged

Properly reset the EmbeddedSwingComposite when disposed#1231
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:swing-reset-embedded-swing-composite

Conversation

@ptziegler
Copy link
Contributor

This restores the UIDefaults for the fonts when the widget is disposed. Otherwise changes to e.g. the LookAndFeel fonts would be preserved, even when the LookAndFeel dialog has been closed.

Furthermore, disposing the Swing Frame is now done via a DisposeEvent. The dispose() method is not called recursively, so there is no guarantee that it is actually disposed.

@ptziegler ptziegler force-pushed the swing-reset-embedded-swing-composite branch 3 times, most recently from be51701 to 57260a8 Compare September 16, 2025 19:07
@ptziegler ptziegler added this to the 1.22.0 milestone Sep 16, 2025
This restores the UIDefaults for the fonts when the widget is disposed.
Otherwise changes to e.g. the LookAndFeel fonts would be preserved, even
when the LookAndFeel dialog has been closed.

Furthermore, disposing the Swing Frame is now done via a DisposeEvent.
The dispose() method is not called recursively, so there is no guarantee
that it is actually disposed.
@ptziegler ptziegler force-pushed the swing-reset-embedded-swing-composite branch from 57260a8 to 4378ec5 Compare September 16, 2025 19:32
@github-actions
Copy link

Test Results

 2 720 files  ±0   2 720 suites  ±0   10h 1m 17s ⏱️ -11s
 6 989 tests ±0   6 920 ✅ ±0   69 💤 ±0  0 ❌ ±0 
27 956 runs  ±0  27 680 ✅ ±0  276 💤 ±0  0 ❌ ±0 

Results for commit 4378ec5. ± Comparison against base commit a5acbe5.

@ptziegler ptziegler merged commit 0a7b291 into eclipse-windowbuilder:master Sep 17, 2025
9 checks passed
@ptziegler ptziegler deleted the swing-reset-embedded-swing-composite branch September 17, 2025 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0