Skip to content

Fixed restricted users' usernames not being clickable - #5405

Merged
pajlada merged 5 commits into
masterfrom
feat/auto-link-mention-element
Jun 1, 2024
Merged

pajlada merged 5 commits into
masterfrom
feat/auto-link-mention-element

Conversation

@pajlada

@pajlada pajlada commented May 19, 2024

Copy link
Copy Markdown
Member

also made the linking be automatic from now on, ensuring we don't make the same mistake again with MentionElements

@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


MessageElement *MentionElement::setLink(const Link &link)
{
assert(false && "MentionElement::setLink should not be called. Pass "

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: no header providing "assert" is directly included [misc-include-cleaner]

src/messages/MessageElement.cpp:15:

+ #include <cassert>

@Mm2PL

Mm2PL commented May 19, 2024

Copy link
Copy Markdown
Collaborator

also made the linking be automatic from now on, ensuring we don't make the same mistake again with MentionElements

Can you please also check how irc user cards work after this change? I am unable to do it for most of today

@pajlada

pajlada commented May 19, 2024 •

Copy link
Copy Markdown
Member Author

also made the linking be automatic from now on, ensuring we don't make the same mistake again with MentionElements

Can you please also check how irc user cards work after this change? I am unable to do it for most of today

The author is unchanged, both open usercards of the name

          V---- this opens usercard for "pajlada"
13:37 pajlada: xd

Mentions are unchanged, they don't open anything

                       V------ this is not clickable
13:37 pajlada: yo @forsen what's up

I don't have to merge this in right now, so I can let you give it a try as well

@pajlada

pajlada commented Jun 1, 2024

Copy link
Copy Markdown
Member Author

image

@pajlada
pajlada enabled auto-merge (squash) June 1, 2024 10:40

@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


MessageElement *MentionElement::setLink(const Link &link)
{
assert(false && "MentionElement::setLink should not be called. Pass "

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: no header providing "assert" is directly included [misc-include-cleaner]

src/messages/MessageElement.cpp:15:

+ #include <cassert>

@pajlada
pajlada merged commit 2a46ee7 into master Jun 1, 2024
@pajlada
pajlada deleted the feat/auto-link-mention-element branch June 1, 2024 11:07
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