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

change hover boxes to display:none #37

Open
mbutterick opened this issue Dec 29, 2016 · 3 comments
Open

change hover boxes to display:none #37

mbutterick opened this issue Dec 29, 2016 · 3 comments
Assignees

Comments

@mbutterick
Copy link
Collaborator

So they get excluded from in-page searching.

@mbutterick mbutterick self-assigned this Dec 29, 2016
@lexi-lambda
Copy link
Member

Would this make the homepage inaccessible by screen readers?

@mbutterick
Copy link
Collaborator Author

Since @LeifAndersen raised the issue, I’ll let @LeifAndersen decide whether hiding the text from searches is more annoying than the current whack-a-mole.

@LeifAndersen
Copy link
Member

@lexi-lambda brings up a good point. I'll give it a shot and see.

Thanks.

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

3 participants