Skip to content

fix: add missing alt attributes in other kanvas pages#6827

Merged
leecalcote merged 1 commit intolayer5io:masterfrom
Fireentity:enhancement/6757-kanvas-seo-other-pages
Sep 22, 2025
Merged

fix: add missing alt attributes in other kanvas pages#6827
leecalcote merged 1 commit intolayer5io:masterfrom
Fireentity:enhancement/6757-kanvas-seo-other-pages

Conversation

@Fireentity
Copy link
Copy Markdown
Contributor

Description

This PR fixes #6757

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Lorenzo Croce <lorenzo.croce@arenastudios.it>
@l5io
Copy link
Copy Markdown
Member

l5io commented Sep 14, 2025

🚀 Preview for commit 3ad71ea at: https://68c697d35e874f6e60e1cbbe--layer5.netlify.app

@Namanv0509
Copy link
Copy Markdown
Contributor

A proper alt will be helpful to improve SEO.

@Fireentity
Copy link
Copy Markdown
Contributor Author

In this case, these are only graphical elements that should be ignored by screen readers. I looked into best practices for such situations and found that it’s better to use an empty alt attribute.

I’ve read https://www.w3.org/WAI/WCAG21/Techniques/html/H67, which recommends using an empty alt attribute for purely decorative images so that they’re ignored by screen readers.
That’s why I applied this approach here.

If you have another suggestion or see a better solution, I’ll be happy to implement it.

@Namanv0509
Copy link
Copy Markdown
Contributor

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

1 similar comment
@Namanv0509
Copy link
Copy Markdown
Contributor

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

Copy link
Copy Markdown
Contributor

@Siddhi-sahu Siddhi-sahu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@leecalcote leecalcote merged commit ad8bb9e into layer5io:master Sep 22, 2025
4 checks passed
@leecalcote
Copy link
Copy Markdown
Member

Good work, @Fireentity. Thank you, @Siddhi-sahu. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve SEO of Kanvas Page on Layer5

5 participants