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

Add alt attributes to images #10

Open
kosarko opened this issue Nov 18, 2024 · 0 comments
Open

Add alt attributes to images #10

kosarko opened this issue Nov 18, 2024 · 0 comments

Comments

@kosarko
Copy link
Member

kosarko commented Nov 18, 2024

Non-text content should have text-alternative (WCAG 1.1.1)

There are images with no alt tag.

A (maybe too coarse as it ignores [attr.alt]) search across component templates

find ./src/ -type f -name '*.html' -exec bash -c "xmllint --html --xpath '//img[not(@alt)]' {} >/dev/null 2>&1" \; -print

reports these files:

src/static-files/deposit.html
src/static-files/cs/deposit.html
src/app/header/header.component.html
src/app/admin/admin-sidebar/admin-sidebar.component.html
src/app/thumbnail/thumbnail.component.html
src/app/home-page/home-page.component.html
src/app/shared/clarin-item-box-view/clarin-item-box-view.component.html
src/app/shared/comcol/comcol-page-logo/comcol-page-logo.component.html
src/app/logout-page/logout-page.component.html
src/themes/custom/app/login-page/login-page.component.html
src/themes/custom/app/logout-page/logout-page.component.html
src/themes/dspace/app/navbar/navbar.component.html

some of those might be false positives (see the [attr.alt] note above), maybe there's a better way to search for these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant