Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Allow non-lowercase values in font, font-family, Sass variables starting with polaris, Sass/CSS variables containing font #49

Merged
merged 7 commits into from
Apr 3, 2019

Conversation

kaelig
Copy link
Contributor

@kaelig kaelig commented Apr 3, 2019

font-family and font are an exception where developers must have the freedom to specify non-lowercase values. For example: BlinkMacSystemFont.

In the case of design tokens files, some variables should also be allowed to contain non-lowercase names: $polaris and other variables containing font.

`font-family` and `font` are an exception where developers must have the freedom to specify non-lowercase values. For example: `BlinkMacSystemFont`.
@kaelig kaelig requested a review from tmlayton April 3, 2019 22:07
@kaelig kaelig requested a review from tmlayton April 3, 2019 22:51
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@tmlayton tmlayton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a small change to the entry to call out Sass variables containing font, otherwise 👍

Co-Authored-By: kaelig <kaelig@users.noreply.github.com>
@kaelig kaelig changed the title Allow non-lowercase values in font and font-family Allow non-lowercase values in, font, font-family, Sass variables starting with polaris, Sass/CSS variables containing font Apr 3, 2019
@kaelig kaelig merged commit 737841a into master Apr 3, 2019
@kaelig kaelig deleted the allow-font-family-camelcase branch April 3, 2019 23:31
@kaelig kaelig changed the title Allow non-lowercase values in, font, font-family, Sass variables starting with polaris, Sass/CSS variables containing font Allow non-lowercase values in font, font-family, Sass variables starting with polaris, Sass/CSS variables containing font Apr 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants