-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
Add examples of Card.outlined and Card.filled in Playground #224
Comments
Hi @albemala, Yes that is a good suggestion, they should be there as well. I actually had to check, because I was not sure if I might had added them already in a not yet released dev branch, but nope. I had not. Definitely will add them. BTW, when you mentioned this I tested the new If you theme border radius on I will file a Flutter issue about this and I also need to put a warning into the Playground about this. The issue is similar to the |
This presentation is now there, coming in version 7.4.0. I also used it on the Card theme settings page. Just sniped the code from the theme simulator, they now use the same code of course. Screen.Recording.2024-03-12.at.13.41.08.movI also show a component styled version of the This shows how a none default radius, due to the variant theming issue, kills the outlined border on themed I also added a special case to FCS card theme, so it does not create a theme for themed card shape when null or default radius value is used. This way, for that specific case, FCS theming will not "kill" the border by creating a custom shape in the the card theme when M3 default value is used. In current version it does that. So that fix/improvement was also added and tests updated to work with the mod. This way the impact with FCS made card theme is exactly the same as it is in vanilla Flutter ThemeData and its Card theme. Nothing I can do about the SDK Card theming limitation, other than file the theming issue. There are already MANY other similar variant theming issues. I now have two mor of the to file, when including this one. I'll try to file both later today. After filing the issues, I still also need to put one of those "known" Flutter theming issues info icons in the Playground and FCS docs for both of them. The other one is about full screen mode and search bar window getting shape in fullscreen mode, if you want to give the popup version another radius that the default one. I have no idea why trivial issues like this keep creeping into the SDK and its theming. They do test, and a lot. But maybe they also need to visually try different configs in test benches to actually see what happens in different use cases, to know what to cover with the tests. Also noticed: The code in the theme simulator that shows and uses the official M3 demo app, is from an old fork and mod I made to be able to add into the Playground. At the time I sniped the code, the I noticed that the official M3 demo at that point used the I do clearly recall that in a original M3 spec and when the first versions of the Flutter M3 demo app was made, the Card outline M3 spec did use the |
Thanks @rydmike for the prompt response and the addition to the Playground! It's unfortunate that you discovered this theming limitation, but on the bright side, now the Flutter team (or someone else) can work on a fix. Hopefully also on all the other similar issues you've mentioned. Maybe there is a common solution for all of them. I see M3 specs are being changed constantly, and it's not easy to keep up with all of them! I look forward to see this fix landing in the next stable package release, again thanks for working on this! |
I see there are these kind of cards displayed in the Theme simulator, but they are missing from the "Widgets showcase" and "Card" sections. It might be nice to have a few examples there as well.
The text was updated successfully, but these errors were encountered: