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

Is skip-link-focus-fix.js still necessary? #1206

Closed
startover909 opened this issue Sep 19, 2017 · 10 comments
Closed

Is skip-link-focus-fix.js still necessary? #1206

startover909 opened this issue Sep 19, 2017 · 10 comments

Comments

@startover909
Copy link

All the information I can find regarding this script and the bug it's intended to fix are years old. I tested current versions of Chrome FireFox IE11 and Edge and none exhibit the bug. Maybe it was fixed a long time ago? Or does it only affect some very old versions of browsers with very few users (and I'd imagine that even fewer keyboard-only users would choose to use such browsers). So is it safe to let this go at this point of time in 2017?

@samikeijonen
Copy link
Contributor

It's still an issue in IE11.

@startover909
Copy link
Author

startover909 commented Sep 21, 2017

Hmmm, I can't reproduce it on IE 11 in Windows 10 v.1703. My IE version is 11.608.15063.0 with update version 11.0.46 (KB4036586). Skip to content link then tab key worked as intended without the fix script. Maybe I'm running a different theme that's somehow different.

@samikeijonen
Copy link
Contributor

On WIN8/IE11 I can still see this issue. My computer wants to update to WIN10 but I try not to:)

@startover909
Copy link
Author

I actually have access to a Win 8.1 system too and the IE version on there is 11.0.9600.18793 with Update Version 11.0.46 (KB4036586), which does not exhibit the issue with. Test site:

https://www.liujack.me

Do you have a link to a site which shows the bug for you so that I can test it as well?

Thanks.

@samikeijonen
Copy link
Contributor

samikeijonen commented Sep 23, 2017

I don't see the issue on that site but I don't have time to debug the theme used in there.

However I just tested with Twenty Seventeen, vanilla _s, and couple of my own themes. They all have the issue if I remove the skip-link JS script.

@startover909 Can you test with Twenty Seventeen and vanilla _s theme also?

@davidakennedy
Copy link
Contributor

Hi @startover909! Thanks for creating this issue. That JavaScript is still needed as @samikeijonen noted, the issue persists in IE 11. I tested on Browserstack with Windows 10 and IE 11 and Windows 8.1 and IE 11, and it's still a problem.

You should see the issue if you remove the script and test. The focus will not move to the content area when you click the skip to content link after "tabbing" to it. Instead it goes back tot he top of the page.

I'm going to close this issue, but feel free to comment again if I'm missing something.

@RetroGameTalk
Copy link

Well it's a dequeue for me.

@westonruter
Copy link

In Core #54421 I've proposed omitting the skip link focus fix by default, given that IE11 only accounts for 0.5% of browser market share. As such, it's not warranted to include wasted script bytes on every single page when barely anyone is using IE11 and even a smaller percentage of them will need the a11y fix. See WordPress/wordpress-develop#1904.

@westonruter
Copy link

Note: I just committed the removal/disabling of the skip-link-focus-fix in core: https://core.trac.wordpress.org/changeset/55861

This should be re-opened.

@Ismail-elkorchi
Copy link
Contributor

The skip-link-focus-fix was already removed in #1424.

kadamwhite added a commit to wikimedia/interconnection-wordpress-theme that referenced this issue May 7, 2024
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

No branches or pull requests

6 participants