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

Divider color too harsh when using system color scheme #255

Closed
ThatOneCalculator opened this issue Dec 30, 2022 · 11 comments
Closed

Divider color too harsh when using system color scheme #255

ThatOneCalculator opened this issue Dec 30, 2022 · 11 comments
Labels
area:ui Issues related to user interface and experience bug Something isn't working

Comments

@ThatOneCalculator
Copy link
Member

ThatOneCalculator commented Dec 30, 2022

The problem

This may be a nitpick, but with no matter what color scheme I try from my wallpaper, the bar that separates notes is just too contrasted from the background.

Screenshots

Default color scheme (good)

System color scheme (bad)

App info

version 0.0.1

moe.craftplacer.kaiteki
versionCode 1672320354

targetSdk 33
minSdk 18

Installed: Dec 26, 2022; 10:53 AM
Updated: Dec 29, 2022; 8:12 PM
@Craftplacer
Copy link
Member

I noticed it myself, and it even looks normal on the M3 specs, so I think it's a bug with Flutter how it decides the dividerColor or outlineVariant

@ThatOneCalculator
Copy link
Member Author

Do you think you could fix that yourself, or override it with another color? Or is this something that has to be PR'd to flutter :(

@Craftplacer
Copy link
Member

We already have an extension method called applyGeneralChanges which are Material theme tweaks that Kaiteki does, so maybe we could see how to adjust the colors to our liking.

@ThatOneCalculator ThatOneCalculator added area:ui Issues related to user interface and experience platform:android This issue targets Android-only nitpick labels Dec 30, 2022
@Craftplacer Craftplacer added bug Something isn't working and removed nitpick labels Dec 30, 2022
@Craftplacer Craftplacer changed the title Note separator color too harsh when using system color scheme (MY) Divider color too harsh when using system color scheme Jan 10, 2023
@Craftplacer
Copy link
Member

Found a related remote issue

material-foundation/flutter-packages#333

@ThatOneCalculator
Copy link
Member Author

Good news? material-foundation/material-dynamic-color-flutter#58 (review)

You could also cherry pick that in, yeah?

@Craftplacer
Copy link
Member

I prefer waiting for it to be merged.

@Craftplacer Craftplacer removed the platform:android This issue targets Android-only label Jan 17, 2023
@ThatOneCalculator
Copy link
Member Author

Reopening. This is from the latest build

Screenshot_20230129-223905.png

version 0.0.1

moe.craftplacer.kaiteki
versionCode 1675044174

targetSdk 33
minSdk 21

Installed: Dec 26, 2022; 1:53 AM
Updated: Jan 29, 2023; 10:32 PM

@tmaihoff
Copy link

tmaihoff commented Jan 31, 2023

What about material-foundation/material-dynamic-color-flutter#58 ? That pull request fixes the issue by adding outlineVariant.

@chaudharydeepanshu
Copy link

It's merged.

@Craftplacer
Copy link
Member

It's merged.

Thanks for your fix 🙏, will update the package

@Craftplacer
Copy link
Member

Issue fixed for real now ❤️

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:ui Issues related to user interface and experience bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants