-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Tango: ClubTwist theme is default, remove Classic theme #2647
Conversation
I must have missed the PR where you added the ClubTwist color scheme. It looks essentially the same. As far as I can tell, the only difference is swapping which decks are blue and which are purple? |
Yup, colors of deck 1&2 are purple (2 decks is the most used layout 2 guess) which is bit easier on they eyes in my experience. |
I might also make decks 3&4 green (like in LateNight?) and entirely remove the Deere'ish blue as it is too pale IMO and is really annoying me over the years. |
This now conflicts with master. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you. Does someone insist on keeping the classic theme as non-default colorscheme (I don't)? What happens when users that use Tango upgrade from 2.2? Will it switch over to ClubTwist automatically?
dlgprefinterface.cpp takes care of that, the first (or the main) theme is selected. |
Apparently not. I'll go ahead and merge now. |
Thx!
Didn't happpen, yet, as I'm with LateNight Flip and medium decks when I play for real. |
As porposed earlier, I remove the Classic scheme from Tango as it was derived from and too similiar to Deere. ClubTwist scheme fits the intended use case much better anyway.