We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I intend to come back to this issue and submit a pull request if I have time.
I see that there is an outstanding issue for this in the upstream library dimsemenov/PhotoSwipe#1503
We have the opportunity in this addon to fix this. The code is pretty straight forward. Within the empty button we include something like
<span class="sr-only" aria-hidden="true">Close slideshow</span>
And then in css
.sr-only { display: none; }
As I say--I'll circle back to this if I get time.
The text was updated successfully, but these errors were encountered:
Also we probably need to support alt tags too. dimsemenov/PhotoSwipe#1423
alt
Sorry, something went wrong.
Instead of the sr-only <span> we can do this:
sr-only
<span>
<button aria-label="Next"></button>
No branches or pull requests
I intend to come back to this issue and submit a pull request if I have time.
I see that there is an outstanding issue for this in the upstream library
dimsemenov/PhotoSwipe#1503
We have the opportunity in this addon to fix this. The code is pretty straight forward. Within the empty button we include something like
And then in css
As I say--I'll circle back to this if I get time.
The text was updated successfully, but these errors were encountered: