cds-snc / platform-forms-client

NextJS application that serves the public-facing website for Forms
https://forms-staging.cdssandbox.xyz/
MIT License
28 stars 12 forks source link

chore: add loading state to submit button #3841

Closed thiessenp-cds closed 1 week ago

thiessenp-cds commented 1 week ago

Summary | Résumé

The original issue was the Agree button would toggle between disabled and back to enabled when the Agree form was submitted.

This fix adds:

Pages updated to use these buttons:

Demo of impacted pages https://github.com/cds-snc/platform-forms-client/assets/107579368/fc0c7491-90f1-4119-954d-4f7ccd530614

Todo: Add to remain auth related pages?

github-actions[bot] commented 1 week ago

:test_tube: Review environment

https://t6g6hifkba2e4cebqtm4pwatum0lhmio.lambda-url.ca-central-1.on.aws/

thiessenp-cds commented 1 week ago

Here's a quick test in Edge. Nothing out of the ordinary :)

https://github.com/cds-snc/platform-forms-client/assets/107579368/4c22395b-a415-4004-8e51-26fcd6095b79