-
Notifications
You must be signed in to change notification settings - Fork 1
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 audit and update #61
Comments
@mestradanu, I think we should update the current Widgets template part to use If we update the "Content Columns" meta fields from the current WYSIWYG that contains both the Lemme know your thoughts. |
My follow up to our offline chat as well: The current PR #66 adds a check for the content to to determine the counter to determine the aria label but the info-theme/template-parts/widgets.php Lines 27 to 43 in 44df5e8
|
@mestradanu made updates on PR #66 |
I did a deep dive recently into the theme that refactored a lot of markup and therefore the theme could use with an accessibility audit and markup changes where needed.
Please branch off
develop
and tag this issue when doing a PR.Will include this with the next release that will include #60 so that we can integrate it into the NCU project.
The text was updated successfully, but these errors were encountered: