Skip to content

Fix exception thrown when unfocusing multiline TextInput#441

Merged
tom-un merged 1 commit intomicrosoft:masterfrom
tido64:tido/fix-multiline-textinput-redbox
Jun 16, 2020
Merged

Fix exception thrown when unfocusing multiline TextInput#441
tom-un merged 1 commit intomicrosoft:masterfrom
tido64:tido/fix-multiline-textinput-redbox

Conversation

@tido64
Copy link
Copy Markdown
Member

@tido64 tido64 commented Jun 10, 2020

  • I am removing an existing difference between facebook/react-native and microsoft/react-native-macos 👍
  • I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos 👍
  • I am making a fix / change for the macOS implementation of react-native
  • I am making a change required for Microsoft usage of react-native

Summary

-resignFirstResponder is not supposed to be called directly. See https://developer.apple.com/documentation/appkit/nsresponder/1532115-resignfirstresponder?language=objc

Resolves #432

Changelog

[macOS] [Fixed] - Exception thrown when unfocusing multiline TextInput

Test Plan

  1. Open RNTester and go to the TextInput screen
  2. Find a multi-line TextInput and try focusing/unfocusing it several times
Microsoft Reviewers: Open in CodeFlow

@tido64 tido64 requested a review from tom-un as a code owner June 10, 2020 09:51
@tido64 tido64 force-pushed the tido/fix-multiline-textinput-redbox branch from 7489158 to a1c615c Compare June 12, 2020 09:33
@ghost ghost removed the Needs: Author Feedback label Jun 12, 2020
@tom-un tom-un merged commit b3543bd into microsoft:master Jun 16, 2020
@tido64 tido64 deleted the tido/fix-multiline-textinput-redbox branch June 16, 2020 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Multiline textInput throws exception when loosing focus

2 participants