Skip to content
This repository has been archived by the owner on Dec 1, 2019. It is now read-only.

Include skip link JS for IE. #937

Merged
merged 5 commits into from
Oct 28, 2019
Merged

Include skip link JS for IE. #937

merged 5 commits into from
Oct 28, 2019

Conversation

byalextran
Copy link

Fixes #935.

Admittedly, I don't have a Windows machine/IE to test this on. However, it seemed like a pretty straightforward fix. Hopefully someone with IE can confirm the fix.

@carolinan
Copy link
Contributor

Please see WordPress/twentynineteen#47 for the expected solution.

@byalextran
Copy link
Author

ugh, sorry about that. PR updated.

you literally linked me to the solution and labelled it "solution," and i somehow still missed it. 🤦🏻‍♂️ saw the changes in link with background information and assumed that was the solution.

Copy link
Contributor

@ianbelanger79 ianbelanger79 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

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

Successfully merging this pull request may close these issues.

Add IE skip link focus fix
3 participants