Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

CSS text-rendering style causes severe performance issues with long strings #1989

Closed
mbpatel82 opened this issue Mar 21, 2015 · 3 comments
Closed

CSS text-rendering style causes severe performance issues with long strings #1989

mbpatel82 opened this issue Mar 21, 2015 · 3 comments
Assignees
@marcysutton
Labels
for: internal contributor The team will address this issue and community PRs are not requested. type: bug

Comments

@mbpatel82
Copy link

https://github.com/angular/material/blob/master/src/core/style/structure.scss#L39

The text-rendering:optimizeLegibility CSS style is applied globally to html and body tags, resulting in severe performance issues on pages that try to render SVGs with large text content. If you need this CSS style, it should only be applied to the elements that need it, not globally.

To see the performance issue, you can reproduce at https://jsfiddle.net/saLmfxza/. Remove the text-rendering CSS style to see the (30x) difference.
@jelbourn jelbourn added type: bug for: internal contributor The team will address this issue and community PRs are not requested. labels Mar 21, 2015
@jelbourn
Copy link
Member

Related #1442

@marcysutton marcysutton self-assigned this Mar 21, 2015
marcysutton pushed a commit that referenced this issue Mar 24, 2015
perf(): relegate global text-rendering style
4343972 
Plus CSS cleanup and comments

Closes #1989
@marcysutton marcysutton mentioned this issue Mar 24, 2015
Closed
@ACDN
Copy link

ACDN commented Mar 25, 2015

Taking out that CSS makes a huge difference in performance. Looking forward to when all the CSS is optimized!

@marcysutton marcysutton mentioned this issue Mar 30, 2015
Closed
@mpiasta-ca
Copy link

Good find 👍

ThomasBurleson added a commit that referenced this issue Jul 6, 2015
@ThomasBurleson
fix(performance, css): disable font aliasing and text-rendering
a9bc534 
Disable global font-smoothing and text-rendering
See article https://bocoup.com/weblog/text-rendering/

Refs #1989. Refs #2701.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@marcysutton marcysutton

Labels
for: internal contributor The team will address this issue and community PRs are not requested. type: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mpiasta-ca @jelbourn @marcysutton @ACDN @mbpatel82