-
Notifications
You must be signed in to change notification settings - Fork 0
feat: liquid flutter 22 #14
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
Conversation
… and German to be less technical
…when the modal has not dismounted yet.
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.
Pull Request Overview
This PR updates the project to support liquid_flutter 22 by revising dependency versions and overrides, refactoring SEC UI widgets with improved exception handling, and updating localization strings. Key changes include:
- Updating dependency versions and adding dependency overrides in pubspec.yaml.
- Refactoring SEC widget logic by replacing the internal _ScanningView with a dedicated ScanningView widget and introducing a new _SecExceptionMapper.
- Updating localization strings in both English and German along with adjustments in SEC-related exception handling and virtual strategy commands.
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
pubspec.yaml | Modified dependency versions and overrides for several packages |
lib/src/ui/sec_widget.dart | Refactored widget structure and enhanced exception mapping |
lib/src/ui/sec_modal.dart | Added delay in close handling and fixed modal properties |
lib/src/ui/scanning_view.dart | Introduced a dedicated scanning view with updated UI behavior |
lib/src/ui/scanning_instruction.dart | Changed gradient colors from theme.background to theme.surface |
lib/src/ui/l10n/sec_locale_en.dart | Updated localization strings (removed extra "with" text) |
lib/src/ui/l10n/sec_locale_de.dart | Updated localization strings with revised message format |
lib/src/sec_reader_exception.dart | Changed SecReaderException to implement Exception instead of Error |
lib/src/sec_reader.dart | Adjusted error handling and expanded platform support (kIsWeb) |
example/* | Updated examples to align with new dependency versions and strategies |
Comments suppressed due to low confidence (1)
lib/src/ui/l10n/sec_locale_de.dart:77
- The word 'Intenetverbindung' appears to be misspelled; consider updating it to 'Internetverbindung'.
String readingsLeft(Object n) { return 'Intenetverbindung in $n Messungen notwendig'; }
No description provided.