fix: add form submission and reset behavior to Button #3603
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
#3534 describes an issue where a
FASTButton[type=submit]
does not actually submit the parent form. In fixing this, I also noticed that aFASTButton[type=reset]
does not reset the form. This introduces a fix to both behaviors.A
FASTButton[type=reset]
will simply invoke thereset()
method of the associated form element when clicked.Submission behavior is a bit more complicated. The
FASTButton[type=submit]
will try to use therequestSubmit()
method of the form if it exists, supplying the method with the button's proxy element as the submitting element (see #3534) for more info. If therequestSubmit()
method is unavailable (Safari) then the proxy'sclick()
method will be invoked.closes #3534
Motivation & context
Issue type checklist
Is this a breaking change?
Adding or modifying component(s) in
@microsoft/fast-components
checklistProcess & policy checklist