Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Force the selection when composition is resolved #2094

Closed
wants to merge 1 commit into from

Conversation

robbertbrak
Copy link
Contributor

Fixes #2093.

@fabiomcosta
Copy link
Contributor

@robbertbrak for this one, we might want to do the same logic that we do at: https://github.com/facebook/draft-js/blob/master/src/component/handlers/edit/editOnSelect.js#L41-L58
I'll see if I'm able to repro the reported issue.

@fabiomcosta
Copy link
Contributor

fabiomcosta commented Jun 11, 2019

@robbertbrak I actually just double checked and even on the scenario that you specified needsRecovery is false, so the logic I pointed out won't work and your suggested change is what we should be using.
I made sure this is all fine by also testing it on Android.
Your change LGTM.
cc @claudiopro

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mrkev has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link

@mrkev merged this pull request in 3be4d37.

vilemj-Viclick pushed a commit to kontent-ai/draft-js that referenced this pull request Jul 16, 2020
Summary:
Fixes facebookarchive#2093.
Pull Request resolved: facebookarchive#2094

Reviewed By: claudiopro

Differential Revision: D21770987

Pulled By: mrkev

fbshipit-source-id: f198a89ce2eae69328daa5ce917085ce0d726dba
alicayan008 pushed a commit to alicayan008/draft-js that referenced this pull request Jul 4, 2023
Summary:
Fixes facebookarchive/draft-js#2093.
Pull Request resolved: facebookarchive/draft-js#2094

Reviewed By: claudiopro

Differential Revision: D21770987

Pulled By: mrkev

fbshipit-source-id: f198a89ce2eae69328daa5ce917085ce0d726dba
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Selection breaks when Chinese composition ends without changes in IE11
4 participants