8000 Jump to the first navigation error · Issue #12128 · wagtail/wagtail · GitHub
[go: up one dir, main page]

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

Jump to the first navigation error #12128

Open
thibaudcolas opened this issue Jul 16, 2024 · 2 comments · May be fixed by #12739
Open

Jump to the first navigation error #12128

thibaudcolas opened this issue Jul 16, 2024 · 2 comments · May be fixed by #12739

Comments

@thibaudcolas
Copy link
Member

Is your proposal related to a problem?

From feedback by blind and partially sighted users:

Error notifications wouldn’t explain what the error actually was, but rely on the user to scroll and find the red highlighted box

Describe the solution you'd like

We could change the error banner to have a button to "Jump to the first error"? This would alleviate some of those difficulties while being heavily compatible with existing error management components.

Current UI for reference:

wagtail form error

Describe alternatives you've considered

It seems to me like more fleshed-out error summaries are incompatible with the "auto-save" interaction model, and our "minimap" jump-menu component does provide a way to avoid scrolling through the page.

I think "Jump to the first error" would be a good fit for Wagtail as it alleviates the problems described here until auto-save happens without requiring a lot of work. And once auto-save is implemented, it will remain useful for CMS forms that don’t have it set up.

Working on this

Anyone can contribute to this. View our contributing guidelines, add a comment to the issue once you’re ready to start.

@Srishti-j18
Copy link
Contributor

I am working on it.

@Srishti-j18
Copy link
Contributor

I have added a PR for this issue. Please let me know if any changes are required.

#12739

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To do
Development

Successfully merging a pull request may close this issue.

2 participants
0