Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Revert "[text_input] introduce TextInputControl" #113724

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

Merged
merged 1 commit into from
Oct 19, 2022

Conversation

zanderso
Copy link
Member

Reverts #76072

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

superlistapp/super_editor#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
Copy link
Contributor

@justinmc justinmc left a comment

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
Member Author

cc @jpnurmi

@zanderso
Copy link
Member Author

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

@justinmc
Copy link
Contributor

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

@jpnurmi
Copy link
Member

jpnurmi commented Oct 20, 2022

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