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

Text rendering fix for Android (and arguably for Windows) #132

Closed
wants to merge 1 commit into from

Conversation

rspeer
Copy link

@rspeer rspeer commented Aug 31, 2011

I understand that your project doesn't officially support WebKit on Android as a browser, but it's a shame to have Bootstrap sites be garbled on Android when they don't have to be. I've found the CSS rule that was causing failure on Android (text-rendering: optimizeLegibility) and commented it out.

As a side effect of removing this line, I have found that fonts on my site render much better on Windows. The "optimizeLegibility" declaration was causing Windows to adopt an aggressive hinting strategy that actively changed the heights of some glyphs, giving text a haphazard appearance.

Of course, you should weigh this benefit against whatever motivation you had to include that rule.

@mdo
Copy link
Member

mdo commented Aug 31, 2011

I'm in favor of removing this and have done so for the 1.2 release already. This setting should be reserved for the OS-level (if you want it on OS X, just turn off LCD font smoothing entirely).

@alexgibson
Copy link

+1 for this.

I just posted a comment on the previously closed issue:

#129 (comment)

@mdo
Copy link
Member

mdo commented Sep 2, 2011

Word. Going to close this since the next release is coming soon and we have this in there.

@mdo mdo closed this Sep 2, 2011
@rlemon
Copy link

rlemon commented Apr 4, 2012

This is still an issue on 2.0.2
I'm seeing this on Chrome 18 stable and FireFox 11 on Ubuntu 11.10

@twbs twbs locked and limited conversation to collaborators Oct 22, 2015
@cvrebert cvrebert added the css label Oct 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants