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

adding alt-text for some images and illustrations #916

Merged
merged 6 commits into from
Mar 18, 2021

Conversation

serahkiburu
Copy link
Contributor

@serahkiburu serahkiburu commented Mar 16, 2021

Alt-text for some images suggested by @maneesha during Core Team Accessibility session. Ready for review.

Copy link
Contributor

@ldko ldko left a comment

Choose a reason for hiding this comment

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

Thanks for submitting this, Serah! I have a couple suggested changes regarding the way the colors are given to more strongly align with the colors as shown.

_extras/discuss.md Outdated Show resolved Hide resolved
_extras/discuss.md Outdated Show resolved Hide resolved
Copy link
Contributor

@ldko ldko left a comment

Choose a reason for hiding this comment

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

Thanks a lot for adding alt text for these, @serahrono !

@ldko ldko merged commit 050856b into swcarpentry:gh-pages Mar 18, 2021
zkamvar pushed a commit that referenced this pull request Apr 21, 2023
* adding alt-text for some images and illustrations

* adding alt-text to second grid illustration

* add alt-text for grid 3

* Update _extras/discuss.md

Co-authored-by: Lauren Ko <lauren.ko@unt.edu>

* Update _extras/discuss.md

Co-authored-by: Lauren Ko <lauren.ko@unt.edu>

* editing down number of alt-text words to the recommended 10-15 

https://kb.iu.edu/d/arwg#guidelines

Co-authored-by: Lauren Ko <lauren.ko@unt.edu>
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

Successfully merging this pull request may close these issues.

2 participants