Skip to content

fix: show streamlink message in selected window - #7231

Merged
pajlada merged 3 commits into
Chatterino:masterfrom
4rneee:fix/streamlink-in-popup
Sep 6, 2026
Merged

pajlada merged 3 commits into
Chatterino:masterfrom
4rneee:fix/streamlink-in-popup

Conversation

@4rneee

@4rneee 4rneee commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Previously the message was always shown in the main window, even when opening something in streamlink while focusing a popup.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

Comment thread src/util/StreamLink.cpp Outdated
Comment thread src/util/StreamLink.cpp Outdated
Comment thread src/util/StreamLink.cpp Outdated

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

Comment thread src/util/StreamLink.cpp
->getMainWindow()
.getNotebook()
.getSelectedPage());
SplitContainer *currentPage = getApp()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: pointee of variable 'currentPage' of type 'SplitContainer *' can be declared 'const' [misc-const-correctness]

Suggested change
SplitContainer *currentPage = getApp()
SplitContainer const*currentPage = getApp()

@Nerixyz

Nerixyz commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

I think this PR itself is fine, but I do wonder what we should do if Chatterino is embedded. There, we still create a main window, but that's not shown. Maybe there should be a last selected split?

@pajlada
pajlada enabled auto-merge (squash) September 6, 2026 10:13
@pajlada
pajlada merged commit 36f55a8 into Chatterino:master Sep 6, 2026
19 of 20 checks passed
babicekmarek pushed a commit to babicekmarek/streamally-chatterino that referenced this pull request Sep 14, 2026
Reviewed-by: pajlada <rasmus.karlsson@pajlada.com>
@4rneee
4rneee deleted the fix/streamlink-in-popup branch September 14, 2026 19:36
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.

3 participants