Dat Vu
Dat Vu
> please rebase it too. Thanks Done. Please review again. @hoangdat
> @dab246 i think all of your comments were addressed, can i rebase and clean the commit history ? Yes, I have closed the previous comments and reviewed them again....
- [ ] Please fix build CI failed
> > * [ ] Please fix build CI failed > > * i am quite confused about the first three errors (`The getter 'pendingFileInfo' isn't defined for the type...
IMO, detecting it from file is most accurate
We will use [UriData](https://api.flutter.dev/flutter/dart-core/UriData-class.html) of `dart:core`. It will only take less memory to store at the time of parsing.
- Speller check not work `HtmlElementView` of Flutter Web but it work on html native web of Safari
> > * Speller check not work `HtmlElementView` of Flutter Web but it work on html native web of Safari > > Screen.Recording.2024-09-10.at.11.47.15.mov > It works fine? What you see...
> > What you see underlined is not `Speller check` > > Screen.Recording.2024-09-10.at.13.22.31.mov > The browser says it is spelling though. I think you should take a closer look. Use...
> This is an issue of LanguageTools https://forum.languagetool.org/t/lt-doesnt-work-on-latest-safari/10421 Please research other Speller check extensions for comparison.