Skip to content

Comments

Refactor code and remove unused imports#164

Closed
MixColumns wants to merge 1 commit intoastarub:masterfrom
MixColumns:master
Closed

Refactor code and remove unused imports#164
MixColumns wants to merge 1 commit intoastarub:masterfrom
MixColumns:master

Conversation

@MixColumns
Copy link
Contributor

some small fixes - some refactoring and some unused imports. Also some small updates to readme.

Copy link
Member

@domai-tb domai-tb left a comment

Choose a reason for hiding this comment

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

Please implement a feature before requesting a merge. You can use the Draft PR feature of GitHub to convert your PR into a draft while developing.

Copy link
Member

Choose a reason for hiding this comment

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

Deprecated analysis options are already addressed in #160

: (context, child) {
return MediaQuery(data: MediaQuery.of(context).copyWith(textScaleFactor: 1), child: child!);
return MediaQuery(
data: MediaQuery.of(context).copyWith(textScaler: TextScaler.noScaling),
Copy link
Member

Choose a reason for hiding this comment

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

Good point. This was already adressed within #160 but I forgot to check the migration guide.

Note: https://docs.flutter.dev/release/breaking-changes/deprecate-textscalefactor#examples

Copy link
Member

Choose a reason for hiding this comment

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

Mensa page becomes general improvement in PR #160

Copy link
Member

Choose a reason for hiding this comment

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

See #165 and #160

@MixColumns MixColumns closed this Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants