Skip to content

Fix bug where font-variant-caps: small-caps resulted in c2sc feature #15

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

Merged
merged 1 commit into from
Oct 28, 2020
Merged

Fix bug where font-variant-caps: small-caps resulted in c2sc feature #15

merged 1 commit into from
Oct 28, 2020

Conversation

lunacookies
Copy link
Contributor

This fixes #14. I’ve never written a non-trivial amount of Javascript, and don’t have a Javascript environment on my machine – I just grepped for c2sc and fixed it as best as I could. Please tell me if I’ve made any mistakes!

@Semigradsky
Copy link
Member

There are failed tests

@lunacookies
Copy link
Contributor Author

Oops, my mistake. Tests are green now.

@Semigradsky Semigradsky merged commit ac2f724 into postcss:master Oct 28, 2020
@Semigradsky
Copy link
Member

Thank you!

@lunacookies
Copy link
Contributor Author

Glad I could help! It’s nice to see such a quick response.

@lunacookies lunacookies deleted the fix-small-caps branch October 28, 2020 07:10
jonathantneal pushed a commit to csstools/postcss-preset-env that referenced this pull request Oct 27, 2021
* Updating as per plugin spec

See postcss/postcss-font-variant#15

* Adding all property back

* Updating System UI plugin to be compatible with PostCSS 8

* Removing no longer existing property

* Autoprefixer false doesn't mess with the rest of the file

* Updating with up to spec from image set plugin

* Using last mod since color-mod was rejected and no longer within pack

* Updating main file to be up to PostCSS 8
romainmenke pushed a commit to csstools/postcss-plugins that referenced this pull request Nov 16, 2021
* Updating as per plugin spec

See postcss/postcss-font-variant#15

* Adding all property back

* Updating System UI plugin to be compatible with PostCSS 8

* Removing no longer existing property

* Autoprefixer false doesn't mess with the rest of the file

* Updating with up to spec from image set plugin

* Using last mod since color-mod was rejected and no longer within pack

* Updating main file to be up to PostCSS 8
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

Successfully merging this pull request may close these issues.

Incorrect conversion of small-caps
2 participants