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

Accessibility fixes for advanced search form. #629

Merged
merged 2 commits into from Jan 13, 2015
Merged

Conversation

@doana
Copy link
Contributor

doana commented Jul 31, 2014

Hi all,

Just some small changes for the sake of accessibility:

@zerocrates

This comment has been minimized.

Copy link
Member

zerocrates commented Aug 18, 2014

This looks pretty good.

We're just doing some checking around to make sure this doesn't cause any particularly undesirable styling side-effects, as legend-in-fieldset often does.

zerocrates added a commit that referenced this pull request Jan 13, 2015
Accessibility fixes for advanced search form.
@zerocrates zerocrates merged commit 22aeb5d into omeka:master Jan 13, 2015
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.