Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix beforeinput event bug in Firefox #32452

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nirajan-khatiwada
Copy link

Related to #24358

Fix the beforeinput event to fire correctly during text replacement scenarios in Firefox.

  • BeforeInputEventPlugin.js

    • Update extractBeforeInputEvent function to handle text replacement scenarios.
    • Ensure beforeinput event fires correctly during MacOS accent popup and spellcheck replacement.
    • Modify getNativeBeforeInputChars and getFallbackBeforeInputChars functions to account for text replacement.
  • DOMEventProperties.js

    • Add beforeinput to the list of DOMEventName types.

Related to facebook#24358

Fix the `beforeinput` event to fire correctly during text replacement scenarios in Firefox.

* **BeforeInputEventPlugin.js**
  - Update `extractBeforeInputEvent` function to handle text replacement scenarios.
  - Ensure `beforeinput` event fires correctly during MacOS accent popup and spellcheck replacement.
  - Modify `getNativeBeforeInputChars` and `getFallbackBeforeInputChars` functions to account for text replacement.

* **DOMEventProperties.js**
  - Add `beforeinput` to the list of `DOMEventName` types.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants