Skip to content
This repository was archived by the owner on Dec 11, 2019. It is now read-only.

Rendering with ClearType #6144

Closed
meyesme opened this issue Dec 11, 2016 · 3 comments · Fixed by #6947
Closed

Rendering with ClearType #6144

meyesme opened this issue Dec 11, 2016 · 3 comments · Fixed by #6947

Comments

@meyesme
Copy link

meyesme commented Dec 11, 2016

I'm using Brave in Windows 7. Compared to Firefox and Chrome it looks all a bit blurry in Brave, and I think it's just that I'm accustomed to ClearType and Brave seemingly doesn't use the same ClearType rendering, if at all.

@bsclifton
Copy link
Member

@meyesme are you able to show how it looks via a screenshot? Also, I am curious if you have a higher DPI setting (like UI being show at 125% of actual size, usually set for larger monitors)

@bsclifton bsclifton added needs-info Another team member needs information from the PR/issue opener. OS/Windows labels Dec 12, 2016
@meyesme
Copy link
Author

meyesme commented Dec 13, 2016

I have the DPI setting in Windows at 100%, the monitor resolution is 1920x1200. The screenshots show http://www.heise.de with Brave, Chrome and Firefox (I've added the browser name at the bottom)

front_brave
front_chrome
front_firefox
text_brave
text_chrome
text_firefox

bsclifton added a commit that referenced this issue Jan 31, 2017
…n Windows

This was originally added to fix #4517, #4518, and #4472... but is no longer needed (I confirmed all those situations work).

Fixes #6462
Fixes #6777
Fixes #6144
Fixes #3556
Fixes #2089

Auditors: @bridiver, @bbondy, @darkdh
@bsclifton
Copy link
Member

bsclifton commented Jan 31, 2017

@meyesme we have a potential fix for this which should be coming soon! Would you be interested in helping us test this to verify the issue is fixed? 😄

Test plan

#6947 (comment)

@bsclifton bsclifton added this to the 0.13.2 milestone Jan 31, 2017
bbondy pushed a commit that referenced this issue Jan 31, 2017
…n Windows

This was originally added to fix #4517, #4518, and #4472... but is no longer needed (I confirmed all those situations work).

Fixes #6462
Fixes #6777
Fixes #6144
Fixes #3556
Fixes #2089

Auditors: @bridiver, @bbondy, @darkdh
@luixxiul luixxiul added release-notes/include and removed needs-info Another team member needs information from the PR/issue opener. labels Feb 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants