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

fix: Making alt text less redundant #813

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

CBID2
Copy link
Contributor

@CBID2 CBID2 commented Jan 3, 2024

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    There was repetitive alt text for the icons/links on the website. So I gave them more descriptive names.
  • Why was this change needed? (You can also link to an open issue here)
    According to Deque University, repetitive alt text can confuse screen readers, thus, making it difficult for users with disabilities to navigate this project's website.
  • Other information:
    Closes 🐛 Bug Report: alt text is redundant  #808

Copy link

vercel bot commented Jan 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 3, 2024 4:21am

Copy link

vercel bot commented Jan 3, 2024

@CBID2 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

codecov bot commented Jan 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2e7cfe9) 19.56% compared to head (fd8b7aa) 19.56%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #813   +/-   ##
=======================================
  Coverage   19.56%   19.56%           
=======================================
  Files          62       62           
  Lines        2914     2914           
=======================================
  Hits          570      570           
  Misses       2344     2344           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dartpain dartpain merged commit c5352f4 into arc53:main Jan 3, 2024
9 of 10 checks passed
@dartpain
Copy link
Contributor

dartpain commented Jan 3, 2024

Thank you!
@holopin-bot @CBID2

@dartpain
Copy link
Contributor

dartpain commented Jan 3, 2024

@holopin-bot @CBID2

Copy link

holopin-bot bot commented Jan 3, 2024

Congratulations @CBID2, you just earned a badge! Here it is: https://holopin.io/claim/clqxnvokr09080flal768x89n

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

@CBID2 CBID2 deleted the making-alt-text-less-redundant branch January 3, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Bug Report: alt text is redundant
2 participants