Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

shihaohong
Copy link

Since it makes more sense for this functionality to be exposed as a plugin, I'm reverting its implementation in the engine. Discussion for this is in flutter/flutter#39858, but essentially, platform-specific APIs make more sense in the form of a plugin, since this behavior will not apply to non-Android platforms.

Reverts #11804, #11451, #11441

@shihaohong shihaohong requested a review from HansMuller April 9, 2020 21:26
@auto-assign auto-assign bot requested a review from iskakaushik April 9, 2020 21:27
Copy link

@HansMuller HansMuller left a comment

Choose a reason for hiding this comment

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

LGTM

@shihaohong shihaohong added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 9, 2020
@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite format_and_dart_test has failed. Please fix the issues identified (or deflake) before re-applying this label.

@fluttergithubbot fluttergithubbot removed the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 9, 2020
@shihaohong shihaohong merged commit 63f4090 into flutter:master Apr 10, 2020
@shihaohong shihaohong deleted the revert-gesture-exclusion branch April 10, 2020 03:44
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 10, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 10, 2020
goderbauer pushed a commit to goderbauer/engine that referenced this pull request Apr 16, 2020
…cts (flutter#17613)

* Revert getSystemGestureExclusionRects and setSystemGestureExclusionRects 

* Remove references to removed PlatformChannelTest.java file
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants