Skip to content
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

chore: fix flutter 3 issues #13

Merged
merged 1 commit into from
Aug 31, 2023
Merged

Conversation

maeddin
Copy link
Contributor

@maeddin maeddin commented Aug 24, 2023

No description provided.

@@ -403,7 +403,7 @@ class GlassContainer extends StatelessWidget {
current = ClipOval(child: current);
} else {
assert(borderRadius != null);
current = ClipRRect(borderRadius: borderRadius, child: current);
current = ClipRRect(borderRadius: borderRadius!, child: current);
Copy link
Owner

Choose a reason for hiding this comment

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

let's use conditional/null-aware(??) operators instead of null-assertion(!)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did not want to remove the assertion and therefore a conditional operator was not necessary. As far as the assert should be removed, you can of course use the ?? BorderRadius.zero.

Copy link
Owner

Choose a reason for hiding this comment

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

any reason why the change in pubspec.lock was required? can it be done by changing the versions in pubspec.yaml

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That was my mistake for committing this. I have not changed any versions. The pubspec.lock was apparently just not synchronized with the pubspec.yaml.

@bharat-1809 bharat-1809 changed the title fixes flutter 3.13 issue chore: fix flutter 3 issues Aug 31, 2023
@bharat-1809 bharat-1809 merged commit 8b91761 into bharat-1809:main Aug 31, 2023
1 of 3 checks passed
@bharat-1809
Copy link
Owner

@all-contributors please add @maeddin for maintenance and code

@allcontributors
Copy link
Contributor

@bharat-1809

I've put up a pull request to add @maeddin! 🎉

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.

None yet

2 participants