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

Image alt tags #233

Closed
Inkering opened this issue Jun 17, 2020 · 1 comment · Fixed by #241
Closed

Image alt tags #233

Inkering opened this issue Jun 17, 2020 · 1 comment · Fixed by #241

Comments

@Inkering
Copy link
Contributor

part of a series of accessibility improvements//maintenance taeyoon asked me to work on, I'm going to sort through and make PR's for these where possible

Few of the current pages on the site have images with alt tags. Particular places like class group photos, header images, etc. For specific people pages, I think that it would be nice to add programmatically built alt tags in the jekyll template.

@Inkering
Copy link
Contributor Author

<img src="{{site.baseurl}}/static/img/{{ page.id }}.jpg"/>

<ul id="people-grid">
{% for person in site.people %}
<li class="col-md-2 col-sm-3 col-xs-4 {% for affiliation in person.affiliation %}{{ affiliation.class | downcase | replace: " ", "-" }} {{ affiliation.role | downcase | replace: " ", "-" }} {% endfor %}"><img src="{{site.baseurl}}/static/img/{{ person.id }}.jpg" />

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 a pull request may close this issue.

1 participant