fix: add missing alt attribute to logo for accessibility#516
fix: add missing alt attribute to logo for accessibility#516inchara13kuppal wants to merge 1 commit intodevsecopsmaturitymodel:mainfrom
Conversation
777169a to
2608fd4
Compare
2608fd4 to
1c796b4
Compare
|
This pull request contains 10 changes that has nothing to do with the description. Not saying the changes are not good improvements. But it makes it hard to keep a tidy flow. As for the missing alt, https://www.w3.org/WAI/tutorials/images/decision-tree/ says implies that an empty I'm sorry, I will close this request. It's fine do group similar changes in one request. But please describe them, and consider which files are included in a particular request. |
|
Thank you for the review and the W3C link, @vbakke ! You are completely right - my branch picked up some unrelated formatting/linting changes locally that I shouldn't have pushed into this specific PR. My apologies for muddying the commit history. I agree with the W3C standard regarding alt="" or alt="Home" for the logo. I will scrap this branch, create a clean one with only the 1-line accessibility fix, and open a fresh PR. I'll group those other UI tweaks into a separate, properly described PR later. Thanks for your patience! |
Description
As part of my familiarization with the DSOMM frontend architecture for my GSoC 2026 proposal, I noticed a minor web accessibility (A11y) gap. The main logo was missing a descriptive
altattribute, which is flagged by screen readers and accessibility audits.Changes Made
altattribute to the<img>tag in the logo component.Testing
npm run lintpasses with zero errors.