Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Upgraded to Flutter 2.5.3 to support display of HEIC images on device… #459

Merged
merged 1 commit into from
Nov 1, 2021
Merged

Upgraded to Flutter 2.5.3 to support display of HEIC images on device… #459

merged 1 commit into from
Nov 1, 2021

Conversation

oxtoacart
Copy link

For getlantern/lantern#451.

Upgrading to the latest Flutter adds support for HEIC on devices that support it.

  • All strings are defined using localized message keys like 'my_message_key'.i18n and are defined in en.po.
  • All colors are defined using Hex (not using built-in colors), are defined in colors.dart and are not duplicated
  • All text styles are defined in text_styles.dart and are not duplicated
  • All icons are using the vector resource from Figma (do not use built-in Icons)
  • Repeated code has been factored into custom widgets
  • Layout looks good in both LTR (English) and RTL (Persian) languages
  • If you refactored existing code, have you tested the refactored functionality against the old version to make sure you didn't break anything?
  • Do the tests pass? Consistently?
  • Did this change improve test coverage?
  • Is the code in question being linted? If not, consider adding a linter step to CI. If yes, make sure the linter is happy.
  • Have you logged tickets for related technical debt with the label “techdebt”?

oxtoacart
oxtoacart commented Oct 26, 2021
View reviewed changes
lib/common/ui/custom/text.dart
@@ -231,7 +231,8 @@ class CTextStyle extends TextStyle {
Color? decorationColor,
TextDecorationStyle? decorationStyle,
double? decorationThickness,
String? debugLabel}) {
String? debugLabel,
TextOverflow? overflow}) {
Copy link
Author

Choose a reason for hiding this comment

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

Signature for TextStyle.copyWith added a new parameter, this updates to keep suit.

oxtoacart
oxtoacart commented Oct 26, 2021
View reviewed changes
lib/common/common.dart
@@ -19,7 +19,6 @@ export 'package:flutter_keyboard_visibility/flutter_keyboard_visibility.dart';
export 'package:google_fonts/google_fonts.dart';
export 'package:i18n_extension/i18n_widget.dart';
export 'package:loader_overlay/loader_overlay.dart';
export 'package:pedantic/pedantic.dart';
Copy link
Author

Choose a reason for hiding this comment

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

We used to use unsafe from pedantic, but that's included in async now so we don't need pedantic.

cosmicespresso
cosmicespresso approved these changes Oct 27, 2021
View reviewed changes
Copy link

@cosmicespresso cosmicespresso left a comment

Choose a reason for hiding this comment

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

Works fine for me!

Base automatically changed from ox/issue436 to ox/messaging November 1, 2021 15:25
@cosmicespresso cosmicespresso merged commit f76d960 into ox/messaging Nov 1, 2021
@cosmicespresso cosmicespresso deleted the ox/issue451 branch November 1, 2021 15:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@cosmicespresso cosmicespresso cosmicespresso approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oxtoacart @cosmicespresso