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.

Sign up for GitHub

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

Jump to bottom

Accessibility #317

Merged
merged 7 commits into from
Dec 12, 2023
Merged

Accessibility #317

merged 7 commits into from
Dec 12, 2023

Conversation

jpmckinney
Copy link
Member

  • fix: Add guard around Export.files for development context
  • fix: Edit sitemaps protocol for development context
  • fix: Fix 4.5:1 color ratios on yellow backgrounds
  • fix(admin): Catch exception if settings.SCRAPYD["url"] not configured. Don't update jobs for new publications.
  • fix(views): Fix edge case if country not set (search.html would display a circle around an "empty" letter)
  • fix(i18n): Make required translatable fields blank=False in all languages
  • ci: Add Accessibility workflow

@jpmckinney jpmckinney force-pushed the pa11y branch 6 times, most recently from 032c9b4 to 2cfe393 Compare December 12, 2023 02:51
@jpmckinney
ci: Add Accessibility workflow
3dbc416 
Ignore WCAG2AA.Principle3.Guideline3_2.3_2_2.H32.2 as there is a submit button if JS is disabled.
A keyboard can be used when JS is enabled.
@jpmckinney jpmckinney merged commit f24b3b4 into main Dec 12, 2023
12 checks passed
@jpmckinney jpmckinney deleted the pa11y branch December 12, 2023 02:58
@jpmckinney jpmckinney mentioned this pull request Dec 12, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jpmckinney