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

Add "Player color theme" setting #79

Merged
merged 23 commits into from
Dec 2, 2022
Merged

Add "Player color theme" setting #79

merged 23 commits into from
Dec 2, 2022

Conversation

nt4f04uNd
Copy link
Owner

@nt4f04uNd nt4f04uNd commented Oct 2, 2022

Add a setting to choose how to color player interface.
Now interface is by default colored based on the current song art.

Video
Record_2022-10-02-20-42-36.mp4

Copy link
Collaborator

@Abestanis Abestanis left a comment

Choose a reason for hiding this comment

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

This is an awesome feature, really cool. Thanks for the video showcasing it!
I don't have much time right now, but I quickly looked over the code and only found a small copy and paste mistake, the rest looks good👍.

I think having a System choice which could use the color from the Material You on Android and the accent color on Windows would be a cool addition on top of this, once this is merged. I'll try this, once I have more time.

lib/localization/arb/intl_de.arb Outdated Show resolved Hide resolved
lib/widgets/show_functions.dart Outdated Show resolved Hide resolved
@nt4f04uNd nt4f04uNd force-pushed the feature/color-thief branch from 178af83 to 3088c15 Compare December 1, 2022 23:05
@nt4f04uNd nt4f04uNd mentioned this pull request Dec 2, 2022
@nt4f04uNd nt4f04uNd merged commit 041c918 into 1.0.8 Dec 2, 2022
@nt4f04uNd nt4f04uNd deleted the feature/color-thief branch December 2, 2022 17:56
@nt4f04uNd
Copy link
Owner Author

Phew, finally, merged this!
Been using it for a few months now, only noticed small bugs and fixed them.
Mostly it took me a while to figure out how to make this whole thing work with tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants