Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

use IdealForegroundColorBrush in title bar elements #1611

Merged

Conversation

thoemmi
Copy link
Collaborator

@thoemmi thoemmi commented Oct 25, 2014

When using a bright theme (such as yellow), you can hardly read the elements in the title bar. With this PR I suggest to use IdealForegroundColor where appropriate.

I'm not sure whether I've changed the correct elements. Please review my changes carefully. 😟

Before:
image

After:
image

@punker76
Copy link
Member

@thoemmi for me it's ok...
//CC @flagbug

@flagbug
Copy link
Member

flagbug commented Oct 26, 2014

I like the idea!

@flagbug flagbug added this to the v1.0 milestone Oct 26, 2014
punker76 added a commit that referenced this pull request Oct 26, 2014
…n-title

use IdealForegroundColorBrush in title bar elements
@punker76 punker76 merged commit 1e49915 into MahApps:master Oct 26, 2014
@thoemmi thoemmi deleted the use-IdealForegroundColorBrush-in-title branch October 27, 2014 19:35
@flagbug
Copy link
Member

flagbug commented Oct 28, 2014

Just noticed that this broke the minimize/maximize/close buttons on the clean window:

2014-10-28 19 44 17

@pixelspezi
Copy link

That’s strange, it works for me. I’m going to create a new branch and a new PR. I did it wrong at first by pushing to the master branch of my repository, it’s better to fix that now.

@punker76
Copy link
Member

@flagbug @thoemmi should now fixed with 25d68e7

@thoemmi
Copy link
Collaborator Author

thoemmi commented Oct 28, 2014

@flagbug @punker76 I've grapped the current master branch but cannot reproduce this issue.
image

@thoemmi
Copy link
Collaborator Author

thoemmi commented Oct 28, 2014

I guess @punker76 already fixed it in 25d68e7?

@flagbug
Copy link
Member

flagbug commented Oct 28, 2014

@thoemmi Yep, it's fixed already

flagbug pushed a commit that referenced this pull request Nov 21, 2014
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants