Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

useMaterial3 advice #81

Closed
fredgrott opened this issue Sep 1, 2022 · 2 comments
Closed

useMaterial3 advice #81

fredgrott opened this issue Sep 1, 2022 · 2 comments
Labels
question This issue is a usage question and will be moved to the Discussions section.

Comments

@fredgrott
Copy link

We can change the do not set useMaterial3 to true advice as when those 6 widgets not yet ported get wrapped via the Material widget class you get this nice thing

md3 demo

I just re worked the sdk md3 demo using FCS 6, good job!

@rydmike
Copy link
Owner

rydmike commented Sep 4, 2022

Hi @fredgrott, I'm not sure I followed exactly if this was a question or issue or just info 😄

But yes there is still the elevation issue in Flutter 3.3 mentioned here flutter/flutter#107190, and yes the issue can be avoided by wrapping problematic widgets, like e.g. PopupMenuButton and Material in a theme that uses Material 2. It is discussed in the issue as a potential tedious workaround.

Also in master channel there is now a change in the Material M3 implementation that will mostly solve the elevation in M3 for widgets that do not yet implement their M3 style and use Material, but it is not yet in stable channel (Flutter 3.3.0), only in master flutter/flutter#110624, and in fact the PR is not even merged into master yet.

Yesterday I got an idea on how I could temporarily fix the themes produced by FCS, so that its M3 dialogs, drawers and popups, get the correct surface tint in M3 mode, even if it is still broken in M3 mode in Flutter 3.3.0.

I will first check what the Flutter core team says on my question regarding when they plan to land fix flutter/flutter#110624, if they have any plan at all to do so in a hotfix.

If it will come in a hotfix to Flutter 3.3, I will probably not implement the idea, but if the fix will not arrive until next stable, maybe Flutter 3.6.0, probably 3-4 months from now. Well then, I will add it to FCS so people can get better M3 themes already now when using M3 with FCS.

When Flutter SDK works better on its own in M3 mode, I will remove the custom theming that fixes the issues. I've done plenty of those in the past already in FCS, that were then removed later when Flutter standard theming started working the same way, thus making my custom fixes no longer needed. Users of FCS don't notice any difference in produce result when I remove such temp fixes.

@rydmike rydmike added the question This issue is a usage question and will be moved to the Discussions section. label Sep 4, 2022
@rydmike
Copy link
Owner

rydmike commented Sep 4, 2022

@fredgrott If this was intended more as an info topic/question, is it OK if I move it to the discussions sections instead of an issue with FlexColorScheme? 😄

Repository owner locked and limited conversation to collaborators Sep 6, 2022
@rydmike rydmike converted this issue into discussion #84 Sep 6, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
question This issue is a usage question and will be moved to the Discussions section.
Projects
None yet
Development

No branches or pull requests

2 participants