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
[framework] Add textField OCR support for framework side #96637
Conversation
@luckysmg Is this still something you are working on? I'm just asking because I'm trying to clean out our review queue. |
Yes,this is still working on, but maybe will not update the progress recently…..I will continue this later😄
发自我的iPhone
…------------------ Original ------------------
From: Ian Hickson ***@***.***>
Date: Wed,May 25,2022 7:26 AM
To: flutter/flutter ***@***.***>
Cc: WenJingRui ***@***.***>, Mention ***@***.***>
Subject: Re: [flutter/flutter] [WIP] Support ios text ocr (PR #96637)
@luckysmg Is this still something you are working on? I'm just asking because I'm trying to clean out our review queue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
The engine side is working around and ready for review. After engine side PR merged, work in framework side(here) will start. |
packages/flutter/lib/src/cupertino/text_selection_toolbar_button.dart
Outdated
Show resolved
Hide resolved
@luckysmg Are you still working on this? Looks like you did good progress since the last time I asked. :-) |
3446bce
to
dbc9306
Compare
HI @Hixie. I come across two things.
|
Hi @justinmc. Would you mind sharing your thoughts on current implementation? O(∩_∩)O |
@tgucio All Done. plz re-review ^_^ |
@@ -25,6 +25,7 @@ | |||
"continueButtonLabel": "جاری رکھیں", | |||
"copyButtonLabel": "کاپی کریں", | |||
"cutButtonLabel": "کٹ کریں", | |||
"scanTextButtonLabel": "متن کو اسکین کریں۔", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this has been updated as follows:
- "scanTextButtonLabel": "متن اسکین کریں۔",
+ "scanTextButtonLabel": "متن کو اسکین کریں۔",
There's an extra word but the Arabic full stop character ('۔') is still there.
canvas.transform(rotationMatrix.storage); | ||
} | ||
|
||
// Draw three lines. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there are still some doubles without decimal point in this method and possibly elsewhere e.g.:
..lineTo(origin.dx, origin.dy + 1)
canvas.drawLine(const Offset(-3, 0), const Offset(3, 0), _painter);
@luckysmg I've LGTM'd with nits: remaining full stop in Urdu l10n and some doubles without decimal point in |
Yes. I ll take a look before merging... Will change. ^_^ Before I didn't understand your meaning very well haha ^_^ |
@tgucio I think all nits done. You can take a look |
Also @justinmc would you mind help to take a look that why google testing still not completed? ^_^ |
Google tests look fine despite the stuck "pending" status. I tried to change the status but it is giving an HTTP error. Merging. |
@luckysmg Thanks for all your work here! |
@luckysmg gj :) |
flutter/flutter@bc49cd1...d55a7d8 2023-07-07 hans.muller@gmail.com Revert "fix a bug when android uses CupertinoPageTransitionsBuilder..." (flutter/flutter#130144) 2023-07-07 engine-flutter-autoroll@skia.org Roll Packages from 771ec9b to 9bcf4bf (12 revisions) (flutter/flutter#130143) 2023-07-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 650ff096488a to 8aa2e6516af1 (1 revision) (flutter/flutter#130116) 2023-07-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 1fa222fae283 to 650ff096488a (3 revisions) (flutter/flutter#130114) 2023-07-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 491f317978f4 to 1fa222fae283 (1 revision) (flutter/flutter#130110) 2023-07-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 48bf7ac59254 to 491f317978f4 (2 revisions) (flutter/flutter#130107) 2023-07-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 42df55a9e63e to 48bf7ac59254 (1 revision) (flutter/flutter#130106) 2023-07-06 5236035+fzyzcjy@users.noreply.github.com Super tiny code optimization: No need to redundantly check whether value has changed (flutter/flutter#130050) 2023-07-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 06c936205d96 to 42df55a9e63e (3 revisions) (flutter/flutter#130100) 2023-07-06 engine-flutter-autoroll@skia.org Manual roll Flutter Engine from bd2e42b203e1 to 06c936205d96 (27 revisions) (flutter/flutter#130097) 2023-07-06 fluttergithubbot@gmail.com Marks Linux firebase_oriol33_abstract_method_smoke_test to be unflaky (flutter/flutter#128398) 2023-07-06 polinach@google.com Next attempt to make inspector weakly referencing the inspected objects. (flutter/flutter#129962) 2023-07-06 82336674+gilnobrega@users.noreply.github.com [a11y] CupertinoSwitch On/Off labels (flutter/flutter#127776) 2023-07-06 henriquenas.dev@gmail.com Add option for flexible space on material SearchDelegate (flutter/flutter#128132) 2023-07-06 engine-flutter-autoroll@skia.org Roll Packages from 7042079 to 771ec9b (12 revisions) (flutter/flutter#130077) 2023-07-06 2539699336@qq.com [framework] Add textField OCR support for framework side (flutter/flutter#96637) 2023-07-06 37002358+fabiancrx@users.noreply.github.com [flutter_tools] modify Skeleton template to use ListenableBuilder instead of AnimatedBuilder (flutter/flutter#128810) 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 bmparr@google.com,rmistry@google.com,stuartmorgan@google.com 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
Congrats to @luckysmg and @tgucio for the shout out in the 3.13 blog post! https://medium.com/flutter/whats-new-in-flutter-3-13-479d9b11df4d |
Preview
12.26.mp4
Related PR in flutter engine (merged):
List which issues are fixed by this PR. You must list at least one issue.
#91896
Pre-launch Checklist
///
).