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.

Sign up for GitHub

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

Jump to bottom

Should not remove arbitrary html-tag-like text #346

Closed
ruyadorno opened this issue Dec 5, 2022 · 0 comments · Fixed by #465
Closed

Should not remove arbitrary html-tag-like text #346

ruyadorno opened this issue Dec 5, 2022 · 0 comments · Fixed by #465
Labels
c: bug Something isn't working

Comments

@ruyadorno
Copy link

A friend posted a cheeky msg that contained a fictional html tag as a way to say hello to their community, see the screenshots below:

Elk UI:

Screen Shot 2022-12-05 at 10 56 32 AM

Mastodon UI:

Screen Shot 2022-12-05 at 10 56 55 AM



I believe the Mastodon UI captured the correct intention here, which was to have a message displaying the tag as plain text. I totally get how it might be tricky to both support html tags and not strip invalid tags, I mostly just wanted to bring this up to other people's attention. If anyone has a strong argument to not support this, feel free to close it 😊

Possibly related to #256
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: bug Something isn't working
Projects
Elk Roadmap
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: don't decode HTML entities (&foo;) until rendering elk-zone/elk
1 participant
@ruyadorno