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

Markdown text in ALT is not rendered #2823

Closed
shuuji3 opened this issue Apr 15, 2024 · 5 comments
Closed

Markdown text in ALT is not rendered #2823

shuuji3 opened this issue Apr 15, 2024 · 5 comments
Labels
c: bug Something isn't working s: pending triage Pending Triage

Comments

@shuuji3
Copy link
Member

shuuji3 commented Apr 15, 2024

Example: https://main.elk.zone/universeodon.com/@elkdev/109709332257865291

Screenshot_20240415-181054.png

@shuuji3 shuuji3 added c: bug Something isn't working s: pending triage Pending Triage labels Apr 15, 2024
Copy link

stackblitz bot commented Apr 15, 2024

Solve in StackBlitz Codeflow Start a new pull request in StackBlitz Codeflow.

@joelanman
Copy link

joelanman commented Apr 27, 2024

I don't think it should be - alt text is for screen reader users primarily, and alt text doesnt support anything other than plain text, encouraging markup doesn't seem helpful. The extra characters would just get read out as asterisks and underscores and so on.

Equally links are not supported in alt text (the actual alt text for the image, not the visible one in elk - but its the same content) and again reading out URLs would not be helpful and is not a description of the image.

@shuuji3
Copy link
Member Author

shuuji3 commented Apr 28, 2024

@joelanman I agree with you. I thought this was a regression as it seems to be rendered in the past but yes, it's better to encourage not to use Markdown in ALT text. I will check if there is still Markdown not-working rendering code used in the past and disable it if it exists.

@shuuji3 shuuji3 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 28, 2024
@joelanman
Copy link

thanks!

@shuuji3
Copy link
Member Author

shuuji3 commented Apr 29, 2024

It looks like the example testing post I found in dev account (not written by myself) was a rejected idea in the past: #1267 So Elk has never implemented Markdown rendering for ALT description in the past and there's no need to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: bug Something isn't working s: pending triage Pending Triage
Projects
None yet
Development

No branches or pull requests

2 participants