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: optimizeLegibility is broken on Android 2.3 #2

Closed
kpeatt opened this issue Jun 18, 2014 · 3 comments
Closed

text-rendering: optimizeLegibility is broken on Android 2.3 #2

kpeatt opened this issue Jun 18, 2014 · 3 comments

Comments

@kpeatt
Copy link
Contributor

kpeatt commented Jun 18, 2014

Not that this is necessarily the biggest concern but this property actively breaks text rendering on Android 2.3 and lower. You end up with what looks like a lot of broken Unicode characters on screen or just missing text.

Some links:

@kennethormandy
Copy link
Owner

This is pretty important! I will have to see if there’s a better way to conditionally use it without relying on trmix, but either way I guess I should drop it. Kerning support on some platforms at the expense of missing text on others isn’t really worth it, especially in a normalizing project. Thanks for the thorough issue report!

kennethormandy added a commit that referenced this issue Jun 21, 2014
@kennethormandy
Copy link
Owner

I haven’t figured out what is best yet, but for now supporting Android 2.3 and maintaining letter-spacing support on Android 4.0 seems more important than adding kerning and basic ligatures on iOS 6 and desktop Safari, so I’ve removed text-rendering.

@kennethormandy
Copy link
Owner

I’m still happy with the decision to remove text-rendering so I’m closing the issue. I don’t want to encourage its use, and I don’t think it’s a property that should have an opinion on OpenType features anyway. Thanks a lot for the heads up on this, @kpeatt!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants