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

RibbonWindow icon is blurry #346

Closed
chdft opened this issue Sep 6, 2016 · 2 comments
Closed

RibbonWindow icon is blurry #346

chdft opened this issue Sep 6, 2016 · 2 comments
Assignees
Labels
Milestone

Comments

@chdft
Copy link

chdft commented Sep 6, 2016

With the current dev-version of Fluent ribbon, the icon of the RibbonWindow is blurry again. A similar behavior was already observed and fixed in #50 and #63, however this time, the blurry icon is not stretched. As you can see in the following image, all icons are 16px tall (as expected).
fluentribbon-icons
(from left to right: Windows Explorer (correctly rendered icon); .Net 4.0; .Net 4.5)

Judging based on the color of the pixels representing the border, it looks like WPF is trying to move the icon half a pixel upwards. Normally this should be prevented by setting SnapsToDevicePixels="True" - as is already done. Maybe setting the Margin="4,1,0,0" instead of Margin="4,0,0,0" helps, like in #63.
I'm currently unable to test this potential fix myself, since I'm still stuck with VS2013 :/

This behavior is reproducible using the Fluent.Test application. (I used the build from appveyor)

Environment

  • Fluent.Ribbon v5.0.0-dev0430
  • Theme Office 2013 (only available)
  • Windows 7 and 10 1607 (Build 14393.10)
  • .NET Framework 4.0 and 4.5
  • DPI-Settings/Scaling 96ppi/100%
@batzen
Copy link
Member

batzen commented Sep 10, 2016

Will have a look at this.

@batzen batzen self-assigned this Sep 10, 2016
@batzen batzen added this to the 5.0.0 milestone Sep 10, 2016
@batzen
Copy link
Member

batzen commented Sep 17, 2016

Changing the margin fixes the issue. Somethings WPF is a bitch...

@batzen batzen closed this as completed in adfb788 Sep 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants