Skip to content

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.

Flutter-Bounty-Hunters/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