-
Notifications
You must be signed in to change notification settings - Fork 193
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
feat: 28841 font synthesis position #691
feat: 28841 font synthesis position #691
Conversation
@dletorey or reviewer, please rename the the PR to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good @dletorey; just one small nit to potentially fix, but I'm approving so as not to hold you up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Congratulations on your first merged pull request. 🎉 Thank you for your contribution! Did you know we have a project board with high-impact contribution opportunities? We look forward to your next contribution. |
🤖 I have created a release *beep* *boop* --- ## [2.2.0](v2.1.0...v2.2.0) (2023-11-03) ### Features * 28841 font synthesis position ([#691](#691)) ([5ced75f](5ced75f)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Description
font-synthesis-position
propertyfont-synthesis
shorthand property to include theposition
valueMotivation
Working on the documentation for [CSS] Add font-synthesis-position property
Additional details
https://drafts.csswg.org/css-fonts-4/#font-synthesis-position
Related issues and pull requests