-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Right now, the evaluate response page shows the options as different separate forms:
platform/src/pages/p/[token]/evaluate-response/index.astro
Lines 45 to 53 in 6f62735
| (['promise', 'denial', 'none'] as const).map((r) => ( | |
| <form class="radio-wrapper" method="POST" action={absUrl(`/p/${token}/evaluate-response/${r}`)}> | |
| <button | |
| type="submit" | |
| class:list={['radio-label', { disabled: r === 'none' && !deadlineExpired }]} | |
| disabled={r === 'none' && !deadlineExpired}> | |
| {t('evaluate-response', `reponse-${r}`)} | |
| </button> | |
| </form> |
However, semantically this should be an option input which submits a single form when clicked/focus is lost. This would require us to explain to users that a context change will occur (https://www.w3.org/TR/2016/NOTE-WCAG20-TECHS-20161007/G13).
But instead, we could also provide a separate submit button altogether (labeled "Continue" or something), which I think is better practice anyway (https://www.w3.org/TR/2016/NOTE-WCAG20-TECHS-20161007/G80). At least was a bit startled after clicking one of the links and immediately starting an analysis.