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 hidden labels on inputs for SR #66

Closed
wants to merge 2 commits into from
Closed

Add hidden labels on inputs for SR #66

wants to merge 2 commits into from

Conversation

Errec
Copy link

@Errec Errec commented Jan 6, 2018

Add hidden label tags for all inputs using the W3C recommendations to fix #53.

@dianaprajescu
Copy link
Contributor

@Errec this looks great. It seems that somehow the files from the dist folder have been deleted. Could you please update the PR and restore them so we can merge it?

Here's a link from Stackoverflow that might help you: https://stackoverflow.com/questions/953481/find-and-restore-a-deleted-file-in-a-git-repository

@m5o
Copy link
Contributor

m5o commented Mar 26, 2018

The edits are nice, but you should use already in-place bootstrap’s hide css styles:

📝 https://getbootstrap.com/docs/4.0/utilities/screenreaders/
📝 https://getbootstrap.com/docs/4.0/utilities/display/#hiding-elements

@stefanneculai
Copy link
Contributor

Thanks for the suggestions on this. We've made quite a few changes lately in this repo. It would be amazing if you could reopen another PR with them.

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

Successfully merging this pull request may close these issues.

A11y Accessibility valid markup
4 participants