flex_color_scheme icon indicating copy to clipboard operation
flex_color_scheme copied to clipboard

Better organization of enabled and disabled chips and rename elevated chips.

Open TahaTesser opened this issue 2 years ago • 4 comments

The current organisation puts all chips into a single section which makes it challenging to distinguish enabled and disabled chips especially when applying custom colors.

Screenshot 2023-09-28 at 16 18 38

Suggestion

Separate chips like this Screenshot 2023-09-28 at 16 24 08

TahaTesser avatar Sep 28 '23 13:09 TahaTesser

Have a bunch of theming features to test and add to Chips as well, so this will be a trivial and logical cleanup of Playground UI when working on those features.

The current design is legacy from when a lot widgets shared the same panel and we wanted to fit them in a compact responsive view using a simple Wrap for all of the different Chip variants. Current design allows for this layout much better and changing to it is indeed a good suggestion.

rydmike avatar Sep 29 '23 19:09 rydmike

Since it is a very responsive layout, this is the best I can do with Wraps. It will still have a 3 col layout mode in some layout sizes, which is not ideal, but just use a bigger screen and you never see that 😃

With Grid and LayoutBuilder I could do better layout that avoids 3 columns, but I don't think it so necessary for this presentation

https://github.com/rydmike/flex_color_scheme/assets/39990307/10d00531-9430-4c1e-aea0-7a88df717840


Q: @TahaTesser what do you mean with rename elevated chips? I'm using the names the Widgets have in Flutter at te moment.


Observation:

I don't think it is still possible to create themes where each chip type Action, Filter, Choice and Input all use from each other different foreground, background colors and disabled color styles, for disabled foreground and background. But I have not checked with all the latest improvements on master channel yet, but last time I looked it was not possible. 😃

rydmike avatar Oct 12 '23 15:10 rydmike

Q: @TahaTesser what do you mean with rename elevated chips? I'm using the names the Widgets have in Flutter at te moment.

I was thinking if it could be better to have names like FilterChip and `Elevated FilterChip' to see which ones are elevated. As the chips blend when you modify elevation, can't disguise which is elevated variant.

TahaTesser avatar Oct 17 '23 11:10 TahaTesser