Skip to content

Conversation

@LouiseHsu
Copy link
Contributor

@LouiseHsu LouiseHsu commented Aug 2, 2023

This PR adds framework support for the Look Up feature in iOS.

Simulator.Screen.Recording.-.iPhone.14.Pro.Max.-.2023-07-11.at.15.23.56.mp4

The corresponding merged engine PR can be found here.
This PR addresses #82907
More details are available in this design doc.

This is the same PR as #130532, this is an attempt to fix the Google Testing issue

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See [testing the engine] for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.

add missing Cupertino localizations

attempt to fix remote tests

fix failing test

fix linux-analyze

rename var, remove unintentionally commited changes

rename var

PR feedback

fix linux analyze

move platform channel check for look up into test from set up

fix tests

restart tests
@LouiseHsu LouiseHsu requested a review from hellohuanlin August 2, 2023 18:37
@github-actions github-actions bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. a: internationalization Supporting other languages or locales. (aka i18n) f: cupertino flutter/packages/flutter/cupertino repository labels Aug 2, 2023
@hellohuanlin
Copy link
Contributor

carrying over the stamp from another PR. can you link that PR in your description?

@LouiseHsu
Copy link
Contributor Author

@hellohuanlin I did! "This is the same PR as #130532, this is an attempt to fix the Google Testing issue"

@hellohuanlin
Copy link
Contributor

oops - i missed that.

@LouiseHsu LouiseHsu added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 2, 2023
@LouiseHsu LouiseHsu marked this pull request as ready for review August 2, 2023 20:10
@auto-submit auto-submit bot merged commit f019789 into flutter:master Aug 2, 2023
@LouiseHsu LouiseHsu deleted the new_lookup_branch branch August 2, 2023 22:29
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 3, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Aug 3, 2023
flutter/flutter@b3f99ff...c00d241

2023-08-03 [email protected] Manual roll Flutter Engine from 9304c0074d29 to 4c1157b9da54 (24 revisions) (flutter/flutter#131830)
2023-08-03 [email protected] Assert against infinite values of control points in CatmullRomSpline (flutter/flutter#131820)
2023-08-02 [email protected] Update stack_frame.dart to parse unexpected error format to null. (flutter/flutter#131786)
2023-08-02 [email protected] Replace TextField.canRequestFocus with TextField.focusNode.canRequestFocus (flutter/flutter#130164)
2023-08-02 [email protected] Mention that the widget tree is not disposed on exit (flutter/flutter#131637)
2023-08-02 [email protected] [framework] Add Look Up to selection controls for iOS  (flutter/flutter#131798)
2023-08-02 [email protected] Add documentation in flutter.groovy noting that we always use the latest available android version (flutter/flutter#131705)
2023-08-02 [email protected] [framework] lerp images in a save layer. (flutter/flutter#131703)
2023-08-02 [email protected] Roll Packages from 3dc00c1 to 4e4961a (1 revision) (flutter/flutter#131788)
2023-08-02 [email protected] add gem dependency to the ios_app_with_extensions_test (flutter/flutter#131713)
2023-08-02 [email protected] Roll Flutter Engine from d6b962d0e36d to 9304c0074d29 (4 revisions) (flutter/flutter#131790)
2023-08-02 [email protected] [flutter_tools] set terminal.singleCharMode to false after attach finishes (flutter/flutter#131723)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@iamsahilsonawane
Copy link

iamsahilsonawane commented Aug 7, 2023

@LouiseHsu Thanks for the commit! will this support "Lookup" option for SelectableText as well?

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

Labels

a: internationalization Supporting other languages or locales. (aka i18n) a: text input Entering text in a text field or keyboard related problems autosubmit Merge PR when tree becomes green via auto submit App f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants