-
Notifications
You must be signed in to change notification settings - Fork 26
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
Feature/4720 accessibility improvements visibility navigation #4776
Feature/4720 accessibility improvements visibility navigation #4776
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4776 +/- ##
=======================================
Coverage 96.55% 96.55%
=======================================
Files 748 748
Lines 25399 25399
Branches 3354 3354
=======================================
Hits 24525 24525
Misses 609 609
Partials 265 265 ☔ View full report in Codecov by Sentry. |
src/openforms/submissions/templates/report/submission_report.html
Outdated
Show resolved
Hide resolved
src/openforms/ui/static/ui/scss/components/skiplink/_skiplink.scss
Outdated
Show resolved
Hide resolved
aa824c3
to
ae57a04
Compare
ae57a04
to
95ce084
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll wait for the SDK PR to be merged and released so we get a proper CI check when this is merged.
Partially Closes #4720
Changes
Checklist
Check off the items that are completed or not relevant.
Impact on features
Release management
I have updated the translations assets (you do NOT need to provide translations)
./bin/makemessages_js.sh
./bin/compilemessages_js.sh
Commit hygiene