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

Allow overriding text-rendering #445

Closed
wants to merge 1 commit into from
Closed

Allow overriding text-rendering #445

wants to merge 1 commit into from

Conversation

dealloc
Copy link
Contributor

@dealloc dealloc commented Jan 7, 2017

Pull Request

Fixes #426

Changes proposed:

  • Add
  • Fix
  • Remove
  • Update

This update adds a variable $body-text-rendering that defaults to optimizeLegibility (current setting) but allows the user to override this setting. The reasoning behind this and the issue this closes can be found #426

@jgthms
Copy link
Owner

jgthms commented Mar 26, 2017

Nice! Can you move that variable to variables.sass?

@dealloc
Copy link
Contributor Author

dealloc commented Mar 26, 2017

I had to duplicate this in #600 because turns out I had accidentally removed my fork.

@dealloc dealloc closed this Mar 26, 2017
@dealloc dealloc mentioned this pull request Mar 26, 2017
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

performance: optimizeLegibility
2 participants