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

Improve accessibility in site profiles #1758

Open
wants to merge 1 commit into
base: devns
Choose a base branch
from
Open

Improve accessibility in site profiles #1758

wants to merge 1 commit into from

Conversation

marcus-herrmann
Copy link

While ProcessWire's core site profiles aren't as central to PW as other core themes to other Content Management Systems ProcessWire should at least inspire developers to make their ProcessWire websites more accessible.

Both "core themes" of WordPress and Drupal (8), Twenty Sixteen and Bartik lead themers by good example, accessibility-wise. ProcessWire should do this as well, in my opinion.

The changes in this merge request do not affect the visual output of the core site profiles, but:

  • their behaviour when using a site profile via keyboard, by adding a "jump to content" bypass block
  • their output towards screen readers, especially regarding the search form (I added labels since placeholders should never be used as form input labels and changed the way of hiding elements)
  • their semantics (main, aside, role-attributes)

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

Successfully merging this pull request may close these issues.

2 participants