benruehl / adonis-ui

Lightweight UI toolkit for WPF applications offering classic but enhanced windows visuals
https://benruehl.github.io/adonis-ui/
MIT License
1.71k stars 143 forks source link

Add DarkEmber color scheme #42

Closed nkristek closed 4 years ago

nkristek commented 4 years ago

Summary

This pull request adds a new color scheme to the existing ones. It uses the color palette from the Dark scheme, but changes the accent colors to variants of orange (from blue).

Please note: This pull request does not include a change to the documentation.

Capture

Motivation and Context

Especially orange fits pretty well for a dark theme and it increases the variety of available color options.

Details

Types of changes

Checklist:

benruehl commented 4 years ago

Thank you for contributing again! :)

In general, additional color schemes are welcome. I am unsure if we should add this particular one though, because it changes the accent color only. Individual colors can be changed by developers easily already. If we add this color scheme, I am afraid that we will have to include more color schemes all being identical except having different accent colors.

I'd be interested in the opinions of others regarding this topic, though.

From a developer's perspective I'd like to keep the number of included color schemes low to reduce necessary testing efforts.

Insire commented 4 years ago

In my opinion this would fit better in a separate repository/ nuget package. Maybe something like adonis-ui-extensions?

While i'm not opposed to having more colors and styles out of the box, i'd prefer to have the option to stay with the 2 default themes and optin to more.

nkristek commented 4 years ago

Thanks for your feedback. After some consideration I will close this PR.