Skip to content

fix(controls): unhook event on closed fluentwindow#1682

Merged
pomianowski merged 2 commits intolepoco:mainfrom
dkuaf:feature/memoryleak-fluentwindow
Apr 17, 2026
Merged

fix(controls): unhook event on closed fluentwindow#1682
pomianowski merged 2 commits intolepoco:mainfrom
dkuaf:feature/memoryleak-fluentwindow

Conversation

@dkuaf
Copy link
Copy Markdown
Contributor

@dkuaf dkuaf commented Apr 7, 2026

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

event is never unsubscribed

Issue Number: #1648
fixes #1648

What is the new behavior?

unsubscribe event on closed

Other information

n/a

@dkuaf dkuaf requested a review from pomianowski as a code owner April 7, 2026 10:43
@github-actions github-actions Bot added controls Changes to the appearance or logic of custom controls. PR Pull request dotnet release labels Apr 7, 2026
Comment thread src/Wpf.Ui/Controls/FluentWindow/FluentWindow.cs Outdated
Comment thread src/Wpf.Ui/Controls/FluentWindow/FluentWindow.cs Outdated
Comment thread src/Wpf.Ui/Controls/FluentWindow/FluentWindow.cs Outdated
Co-authored-by: Nuklon <Nuklon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

controls Changes to the appearance or logic of custom controls. dotnet PR Pull request release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Memory leak in FluentWindow

3 participants