-
Notifications
You must be signed in to change notification settings - Fork 155
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
Document has no main content area #1652
Comments
Closing this since we now have the Vue.js app wrapped in a |
But is it wrapping the main content of the page or everything? |
@pascalwengerter It looks like it is wrapping everything, and this is not how the main element is intended/could be a WCAG violation: web/packages/web-container/index.html.ejs Line 28 in 695d3df
|
@marcus-herrmann sorry, was under the wrong impression that your commit had fixed it. How critical is fixing this for the accessibility certification? @kulmann maybe we need to be solving this on the extension level, currently we're using
|
@pascalwengerter Rather critical. Wrongly applied landmarks are worse than no landmarks at all. I'm quite certain this can be seen as a violation of WCAG SC 2.4.1, Bypass blocks. My fix back in the day fixed this, but another |
No description provided.
The text was updated successfully, but these errors were encountered: