Skip to content

fix: set parent for settings dialog from account switcher - #6543

Merged
pajlada merged 3 commits into
Chatterino:masterfrom
Nerixyz:fix/wl-account-button
Oct 25, 2025
Merged

pajlada merged 3 commits into
Chatterino:masterfrom
Nerixyz:fix/wl-account-button

Conversation

@Nerixyz

@Nerixyz Nerixyz commented Oct 14, 2025

Copy link
Copy Markdown
Contributor

We set the parent of the settings dialog to the account switcher popup when opening the settings from the popup. Since that dialog gets closed when the button is clicked, the settings dialog would close. This is only a problem if the settings aren't opened before the button is clicked on the account switcher, since the parent of the settings dialog only gets set once.

@pajlada
pajlada enabled auto-merge (squash) October 25, 2025 12:24
@pajlada
pajlada merged commit 8621701 into Chatterino:master Oct 25, 2025
18 checks passed
@Nerixyz
Nerixyz deleted the fix/wl-account-button branch October 25, 2025 12:54
Pietro228 pushed a commit to Stream-Ally/streamally-chatterino that referenced this pull request Dec 5, 2025
Pietro228 pushed a commit to Stream-Ally/streamally-chatterino that referenced this pull request Dec 5, 2025
babicekmarek pushed a commit to babicekmarek/streamally-chatterino that referenced this pull request Sep 14, 2026
babicekmarek pushed a commit to babicekmarek/streamally-chatterino that referenced this pull request Sep 14, 2026
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.

2 participants