Skip to content

Revert "[text_input] introduce TextInputControl" - #113724

Merged
zanderso merged 1 commit into
masterfrom
revert-76072-text-input-handler
Oct 19, 2022
Merged

zanderso merged 1 commit into
masterfrom
revert-76072-text-input-handler

Conversation

@zanderso

Copy link
Copy Markdown
Member

Reverts #76072

An apparent bugfix in the Dart SDK is newly revealing that this change broke super_editor in customer_testing.

https://github.com/superlistapp/super_editor/pull/837

Unfortunately, super_editor is broken on Flutter master channel, so we are unable to simply apply the compensating change there and roll a new version into flutter/tests. Since the breakage is only revealed by a new Dart SDK, we are blocked from rolling the Engine into the framework until this is resolved. Since the roll blockage is a P0, we are preferring to solve it by landing a revert.

@flutter-dashboard flutter-dashboard Bot added a: text input Entering text in a text field or keyboard related problems d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos documentation framework flutter/packages/flutter repository. See also f: labels. c: contributor-productivity Team-specific productivity, code health, technical debt. labels Oct 19, 2022
@zanderso
zanderso requested a review from justinmc October 19, 2022 20:49

@justinmc justinmc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@zanderso
zanderso merged commit 9064bf6 into master Oct 19, 2022
@zanderso
zanderso deleted the revert-76072-text-input-handler branch October 19, 2022 22:00
@zanderso

Copy link
Copy Markdown
Member Author

cc @jpnurmi

@zanderso

Copy link
Copy Markdown
Member Author

This can be relanded after flutter/tests#165 lands.

@justinmc

Copy link
Copy Markdown
Contributor

Looks like flutter/tests#165 was landed, @jpnurmi are you able to resubmit your PR #76072?

@jpnurmi

jpnurmi commented Oct 20, 2022

Copy link
Copy Markdown
Member

Thanks, resubmitted: #113758

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

Labels

a: text input Entering text in a text field or keyboard related problems c: contributor-productivity Team-specific productivity, code health, technical debt. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants