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

Conversation

jonahwilliams
Copy link
Contributor

@jonahwilliams jonahwilliams commented Aug 8, 2018

Currently, Android always responds to text editing by setting the text affinity to downstream. Flutter can change the affinity to upstream if it needs to.

The iOS text input sets editing text to an upstream affinity, so newlines don't move the cursor initially. Fixed by setting to always return downstream.

Fixes flutter/flutter#20293

Copy link
Contributor

@liyuqian liyuqian left a comment

Choose a reason for hiding this comment

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

LGTM

@jonahwilliams jonahwilliams merged commit 4208f84 into flutter:master Aug 8, 2018
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 8, 2018
flutter/engine@08b660e...4208f84

git log 08b660e..4208f84 --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08 [email protected] make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08 37626415+[email protected] Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07 37626415+[email protected] Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07 [email protected] Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 8, 2018
flutter/engine@08b660e...4c7e5d5

git log 08b660e..4c7e5d5 --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08 37626415+[email protected] Roll src/third_party/skia 0c5b0b1dd692..9c9611fcc1bb (13 commits) (flutter/engine#5977)
2018-08-08 [email protected] make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08 37626415+[email protected] Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07 37626415+[email protected] Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07 [email protected] Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 8, 2018
flutter/engine@08b660e...78f8bca

git log 08b660e..78f8bca --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08 [email protected] Annotate deprecated methods with @deprecated (flutter/engine#5976)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 0c5b0b1dd692..9c9611fcc1bb (13 commits) (flutter/engine#5977)
2018-08-08 [email protected] make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08 37626415+[email protected] Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07 37626415+[email protected] Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07 [email protected] Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 8, 2018
flutter/engine@08b660e...3777931

git log 08b660e..3777931 --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08 [email protected] Roll Dart to version eab492385c3f345cb2f44f3b702b0e30e4a9c107 (flutter/engine#5979)
2018-08-08 [email protected] Fix IsolateStart event kind. (flutter/engine#5978)
2018-08-08 [email protected] Annotate deprecated methods with @deprecated (flutter/engine#5976)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 0c5b0b1dd692..9c9611fcc1bb (13 commits) (flutter/engine#5977)
2018-08-08 [email protected] make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08 37626415+[email protected] Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07 37626415+[email protected] Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07 [email protected] Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 8, 2018
flutter/engine@08b660e...3501acb

git log 08b660e..3501acb --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08 37626415+[email protected] Roll src/third_party/skia 9c9611fcc1bb..0d5d0659a684 (7 commits) (flutter/engine#5980)
2018-08-08 [email protected] Roll Dart to version eab492385c3f345cb2f44f3b702b0e30e4a9c107 (flutter/engine#5979)
2018-08-08 [email protected] Fix IsolateStart event kind. (flutter/engine#5978)
2018-08-08 [email protected] Annotate deprecated methods with @deprecated (flutter/engine#5976)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 0c5b0b1dd692..9c9611fcc1bb (13 commits) (flutter/engine#5977)
2018-08-08 [email protected] make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08 37626415+[email protected] Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07 37626415+[email protected] Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07 [email protected] Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 9, 2018
flutter/engine@08b660e...91537ab

git log 08b660e..91537ab --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08 [email protected] Revert "Temporarily add travis/analyze.sh back for Chrome bot (flutter#5961)" (flutter/engine#5966)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 9c9611fcc1bb..0d5d0659a684 (7 commits) (flutter/engine#5980)
2018-08-08 [email protected] Roll Dart to version eab492385c3f345cb2f44f3b702b0e30e4a9c107 (flutter/engine#5979)
2018-08-08 [email protected] Fix IsolateStart event kind. (flutter/engine#5978)
2018-08-08 [email protected] Annotate deprecated methods with @deprecated (flutter/engine#5976)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 0c5b0b1dd692..9c9611fcc1bb (13 commits) (flutter/engine#5977)
2018-08-08 [email protected] make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08 37626415+[email protected] Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08 37626415+[email protected] Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07 37626415+[email protected] Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07 [email protected] Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
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.

3 participants