Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Render styling for colored host masks #4235

Merged
merged 1 commit into from
Jun 2, 2021
Merged

Conversation

angerson
Copy link
Contributor

@angerson angerson commented May 22, 2021

On some IRC networks, users have vanity host masks with colors or other text styling.
Rizon is one such network.

For example, a user connecting from 127.0.0.1 could instead have the host
angerson@this.is.my.host.mask. this.is.my.host.mask may have IRC color code
characters in it, which without this change would be displayed as a bunch of jumbled
garbage in the /whois response or join/part messages.

Resolves #4232.

@angerson
Copy link
Contributor Author

angerson commented May 22, 2021

Most of the other components (joins, parts, etc.) also lack this styling. Would the correct thing to do be to wrap every {{ message.hostmask }} with a ParsedMessage? I don't think a more abstract solution is necessary, because it's only hostmasks that are likely to be like this.

Edit: I updated every component I could find.

@angerson angerson changed the title Parse styled text in /whois host mask Render styling for colored host masks May 22, 2021
@MaxLeiter
Copy link
Member

MaxLeiter commented May 26, 2021

Thanks for the PR, @angerson! It seems like there's a small linting error in client/components/MessageTypes/chghost.vue. If you fix that + squash your commits I think it's ready to merge

On some IRC networks, users have vanity host masks with colors or other text styling.
Rizon is one such network.

For example, a user connecting from 127.0.0.1 could instead have the host
angerson@this.is.my.host.mask. this.is.my.host.mask may have IRC color code
characters in it, which without this change would be displayed as a bunch of jumbled
garbage in the /whois response or join/part messages.

Resolves thelounge#4232.
@angerson
Copy link
Contributor Author

Done, thanks @MaxLeiter.

@MaxLeiter MaxLeiter merged commit 6182d23 into thelounge:master Jun 2, 2021
@MaxLeiter MaxLeiter added this to the 4.3.0 milestone Jun 5, 2021
@MaxLeiter MaxLeiter added the Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors. label Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply parseStyle to /whois information
2 participants