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

Look into how to enable alt text syntax for images #80

Open
decentral1se opened this issue Oct 12, 2024 · 7 comments
Open

Look into how to enable alt text syntax for images #80

decentral1se opened this issue Oct 12, 2024 · 7 comments

Comments

@decentral1se
Copy link
Contributor

If I do this in a post:

![my alt text](https://...)

I get:

<p><img loading="lazy" src="https://..."/></p>

The alt="my alt text" doesn't seem to make it through the conversion?

@cblgh
Copy link
Owner

cblgh commented Oct 18, 2024

oh dang, the differences in markdown renderers. it probably never worked in that case!

@cblgh cblgh changed the title Missing alt text on markdown -> img conversion? Look into how to enable alt text syntax for images Oct 18, 2024
@cblgh
Copy link
Owner

cblgh commented Oct 18, 2024 via email

@decentral1se
Copy link
Contributor Author

Nice, yes, that looks promising. Thanks for looking into it.

Issue upstreamed in the meantime: gomarkdown/markdown#318

@decentral1se
Copy link
Contributor Author

Oh nice some source codez gomarkdown/markdown#318 (comment)

@cblgh
Copy link
Owner

cblgh commented Oct 19, 2024

@decentral1se just spun up my dev forum, and i'm having alt text displayed! but not title which is more useful for browsers when assistive tech isn't turned on, unless you do like in gomarkdown/markdown#318 (comment) and have it at the end which i don't think anyone will remember tbh lol

can you give me the exact post input you are using so i can repro the issue you opened this with?

here's what i used which works

a post with some alt text imagery soon

![teh alt text](https://tendigits.space/assets/img/drawtober-2024-19.jpg "Title")
okay nice

and actually the drawtober thread is a good example of alt text being used and actually in the markup! https://forum.merveilles.town/thread/101/drawtober-2024-67/#1887

@cblgh
Copy link
Owner

cblgh commented Oct 19, 2024

@decentral1se i'm now using the alt text to also set the title attribute :) check out e478e28

@decentral1se
Copy link
Contributor Author

oh sheeeyat, that's great stuff! ok lemme do a test on the forum / dev instance and hopefully we can close this one out 🙏 nice detective work & patch!

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

No branches or pull requests

2 participants