-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
36125 add text emphasis position auto #36331
base: main
Are you sure you want to change the base?
36125 add text emphasis position auto #36331
Conversation
Preview URLs (comment last updated: 2024-10-17 10:13:10) |
In this preview, it looks like the "Formal definition" and "Formal syntax" sections still need an update. |
Those can be found in the following Pull Request: |
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.
Thank you for making these updates, @dletorey!
I've added some suggestions and questions
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…deRedDigital/content into 36125-add-text-emphasis-position-auto
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.
Thanks for the updates, @dletorey. They're looking good so leaving a +1.
A few nits need to be resolved before this PR is merged. I'm not seeing the option to apply the suggested changes else I would have committed them.
Update: The writing-mode property can be added to the "See also". And maybe also text-underline-position.
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
Co-authored-by: Dipika Bhattacharya <dipika@foss-community.org>
…deRedDigital/content into 36125-add-text-emphasis-position-auto
Description
auto
value to Valuesauto
into SyntaxMotivation
Working on issue #36125
Additional details
auto
value fortext-emphasis-position
Related issues and pull requests