department-of-veterans-affairs / va-forms-system-core

VA Forms System Core will be a React library hosted in NPM that will allow engineers and designers to easily interact with Forms inside of vets-website.
https://department-of-veterans-affairs.github.io/va-forms-system-core/
6 stars 4 forks source link

VA Forms System Core [Demo Content]: Component choice - select vs. text input #556

Closed TiffanyPender closed 2 years ago

TiffanyPender commented 2 years ago

User Story or Problem Statement

As a user with cognitive issues, it is helpful when simplified choices are made available for form fields that have pre-defined options available.

Details

This may be more to do with form content than the library, but worth documenting - within the Forms Library Core demo form, on step 4: Military Service History, the Branch of Service field is a text input, but could be a select element since there are only a small number of choices available.

Acceptance Criteria

Environment

Steps to Recreate

  1. Visit the Forms Library Core demo form
  2. Navigate to step 4: Military Service History
  3. Verify that Branch of Service field is a text field, not a select field element
taylorkaren commented 2 years ago

The Branch of Service field is a text input on the demo form because that is how it is implemented on the production form that we used as the source and model for the demo form, so that was a form content choice made by team that built the "real" form.

But having said that, we are trying to model good form building accessibility practices, so I think we should go ahead and make this change. The options to include in the select field would be (from What Are the Branches of the US Military?):

obliviga commented 2 years ago

PR is ready for review: https://github.com/department-of-veterans-affairs/va-forms-system-core/pull/566

obliviga commented 2 years ago

Moving this back to In Progress, as the change needs to be applied to the demo in the vets-website repo.

obliviga commented 2 years ago

Verified fix has been deployed: image.png