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

Validate inaccessible pages #285

Open
chipit24 opened this issue Jul 9, 2019 · 0 comments
Open

Validate inaccessible pages #285

chipit24 opened this issue Jul 9, 2019 · 0 comments
Labels

Comments

@chipit24
Copy link
Contributor

chipit24 commented Jul 9, 2019

Related issue: #280.

If the submit button appears on a page that's inaccessible, a user will not be able to complete the form. Consider adding validation that ensures the form can be submitted, by checking that all pages are accessible.


The page only requires one submit button to be valid. However, if this submit button is on, for example, the 2nd page of the form, and there is no navigation button on the 1st page to go to the 2nd page, then the form cannot be submitted and should show an error.

There is currently validation that ensure the form contains at least one submit button; to meet the requirement that this submit button can be accessed, one of two errors can be show:

  1. Show a "Submit button cannot be reached" error if the submit button cannot be accessed by using form navigation buttons.
  2. Show a "Page cannot be reached" error if the user created a page that cannot be accessed using form navigation buttons.

Case 2 above is more generic and probably the better approach to use, as it will also ensure users don't create inaccessible pages.

@tuantruong625 tuantruong625 self-assigned this Jul 11, 2019
@tuantruong625 tuantruong625 added review Review this ticket and removed review Review this ticket labels Jul 11, 2019
@tuantruong625 tuantruong625 removed their assignment Jul 11, 2019
@chipit24 chipit24 added the hold label Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants