-
-
Notifications
You must be signed in to change notification settings - Fork 358
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
fix: add alt tags to landing page images - lighthouse #6803
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
AI-Generated Summary: This pull request includes a fix for the images on the landing page that were missing alt tags. These alt tags have been added in order to improve the accessibility of the site and align it with SEO and web standards best practices. The changes have been made in four files: TheFooter.vue, SearchLanding.vue, ShoppingCartButton.vue, and SearchBar.vue. In summary, all the images have been provided with appropriate alt descriptions. The alt tags are empty where the images are decorative and do not convey content or their absence does not disable users from accessing information. |
Code Climate has analyzed commit a883a52 and detected 0 issues on this pull request. View more on Code Climate. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@thnaylor what's the score now in Lighthouse? |
Accessibility is now in the green mobile&desktop with 91. New:Previous:Comments:I did some quick experiments on the landing page for the other accessibility issues and got 98. So definately some room for improvements. |
pay 50 usd @thnaylor if you can have look on other issues to improve lighthouse! |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
sure, make separate issue on that if so! |
Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type
Context
Before submitting pull request, please make sure:
Optional
Did your issue had any of the "$" label on it?
Community participation
Screenshot 📸
Lighthouse report
Tags used
Copilot Summary
🤖 Generated by Copilot at a883a52
Added or improved
alt
attributes forimg
elements in various components to enhance accessibility and provide meaningful information for screen readers.🤖 Generated by Copilot at a883a52