Skip to content

Bubble onSubmit/onReset behind a feature flag - #19333

Merged
gaearon merged 1 commit into
react:masterfrom
gaearon:form-submit
Jul 13, 2020
Merged

gaearon merged 1 commit into
react:masterfrom
gaearon:form-submit

Conversation

@gaearon

@gaearon gaearon commented Jul 13, 2020

Copy link
Copy Markdown
Collaborator

Same as #19285, but behind a feature flag so it's less of a risk. The flag is off by default, but is adjustable in WWW.

Essentially this is a comeback of #13358. When we tried it last time, it broke a few things internally, and we had to roll back. But it's possible that this code no longer exists. Or maybe we can fix it in the product. So let's give it another try and be more careful this time.

We are doing this now so that we can remove these two events from the special capture phase list. And as a result, fix event replaying for them.

@gaearon
gaearon requested a review from trueadm July 13, 2020 16:09
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jul 13, 2020
@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 771048b:

Sandbox Source
React Configuration

@sizebot

sizebot commented Jul 13, 2020

Copy link
Copy Markdown

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 771048b

@sizebot

sizebot commented Jul 13, 2020

Copy link
Copy Markdown

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 771048b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants