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

Front of House table A11y updates #4490

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

StephDriver
Copy link
Contributor

tables identified via a search for 'table' in the templates within the themes directory.

first commit includes a comment above each table stating what needed fixing, these comments are then removed for the final commit.

For the OLH theme, there were icons that needed updating within the table (as per #4215) which convey information that is relevant to their position within the table, so I updated those too.

closes #4055

@joemull joemull assigned ajrbyers and unassigned joemull Nov 15, 2024
@StephDriver
Copy link
Contributor Author

StephDriver commented Nov 15, 2024

As per discussion this morning, having edited translated text, django-admin commands need to run. Marking this as draft until that's been done.

@StephDriver StephDriver marked this pull request as draft November 15, 2024 10:31
@StephDriver StephDriver assigned StephDriver and unassigned ajrbyers Nov 15, 2024
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.

Table A11y updates
3 participants