Skip to content

fix: show inline twitch gif tooltips without link previews enabled - #7283

Merged
pajlada merged 2 commits into
Chatterino:masterfrom
3r01:fix-inline-twitch-gif-tooltip
Oct 24, 2026
Merged

pajlada merged 2 commits into
Chatterino:masterfrom
3r01:fix-inline-twitch-gif-tooltip

Conversation

@3r01

@3r01 3r01 commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

Twitch gifs as images currently don't have a tooltip if link previews are disabled. This is contrary to how existing inline images (emotes) work in chat. You may want to see the name of the gif without also having link previews enabled.

Fixed by showing tooltips their regardless of the link previews setting, provided the gif is displayed as an inline image.

No change is made to gifs as links.

@Nerixyz

Nerixyz commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

An issue with this is that we're now using the link resolver even though the user didn't enable it.

@3r01

3r01 commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

An issue with this is that we're now using the link resolver even though the user didn't enable it.

Are we? As far as I can see the text comes from the twitch message's placeholder text, there's no link resolver request.

@Nerixyz

Nerixyz commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Oh, I thought this intends to show the tooltip when gifs aren't inline.

@3r01

3r01 commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor Author

No, apologies if my wording wasn't clear. Currently the tooltips only work if the link previews setting is enabled, even when the gif is displayed inline.

I think inline gifs as images should show their tooltip like emotes do, and we already have the data for that. No change is made to gifs displayed as links.

@pajlada
pajlada enabled auto-merge (squash) October 24, 2026 13:34
@pajlada
pajlada merged commit f70eb81 into Chatterino:master Oct 24, 2026
18 checks passed
@3r01
3r01 deleted the fix-inline-twitch-gif-tooltip branch October 24, 2026 14:11
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