search

CuBoulder / express_mono

Monolithic Express Profile - contains all bundles
GNU General Public License v2.0
1 stars 2 forks source link

Disable webform submit button on form submit #877 #878

Closed kevincrafts closed 2 years ago

github-actions[bot] commented 2 years ago

Run Tests When Work Is Completed

Do not merge code before you run tests. Do the following when you have completed work on this PR and all of you reviewers have approved it:

  1. Add the Label "ReadyForTesting".
  2. If the tests pass:
    • Great!
    • Wait for you reviewers to approve the PR.
    • Merge when appropriate.
  3. If the tests fail:
    • You have more work to do!
    • Take the label off.
    • Make your changes.
    • Repeat these steps.
AlanBCole commented 2 years ago

Merging this is. Testing on a locally hosted express site works well.

AlanBCole commented 2 years ago

@jcsparks The code does not changed the submit button text is not changed and the event to disable the button is on submit of the form. The styling change is to add some opacity to the button as well as disabling it.

AlanBCole commented 2 years ago

I merged this code previously as it worked, although I was less than thorough on UI suggestions before accepting this pr. The branch is still around if this is what we want to use for further changes and another pr.