Fix a bug in ClosableDialog

[eivindjahren]

Issue
Resolves #8465

• PR title captures the intent of the changes, and is fitting for release notes.
• Added appropriate release note label
• Commit history is consistent and clean, in line with the contribution guidelines.
• Make sure tests pass locally (after every commit!)

When applicable

• When there are user facing changes: Updated documentation
• New behavior or changes to existing untested code: Ensured that unit tests are added (See Ground Rules).
• Large PR: Prepare changes in small commits for more convenient review
• Bug fix: Add regression test for the bug
• Bug fix: Create Backport PR to latest release

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.66%. Comparing base (aad5994) to head (d009841).
Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8480      +/-   ##
==========================================
- Coverage   90.69%   90.66%   -0.04%     
==========================================
  Files         345      345              
  Lines       20981    20981              
==========================================
- Hits        19029    19022       -7     
- Misses       1952     1959       +7     

Flag	Coverage Δ
gui-tests	75.92% <100.00%> (-0.03%)	⬇️
integration-tests	53.95% <0.00%> (-0.01%)	⬇️
unit-tests	68.66% <0.00%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[andreas-el]

💯 Using the correct name is always a good idea. 💡