-
Notifications
You must be signed in to change notification settings - Fork 519
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
32341 text wrap #2746
32341 text wrap #2746
Conversation
Hi @dletorey, thanks for the PR! I started looking at this, but I didn't find the examples in my local preview at this path "http://127.0.0.1:9090/pages/css/". I am wondering if you also need to do this step, Updating the metadata, so that the examples get added to the page generator. |
I have now added this and yes you are correct. |
Thanks, @dletorey, that fixed it. Yes, based on your BCD PR, I'm testing this in Firefox 124.0b9.
The style inspector also does not show |
Hi @dipikabh This is purely due to
|
Got it! In that case, might be a good idea to make |
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 adding these examples, @dletorey!
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. |
### Description Added `text-wrap-mode` and `text-wrap-style` ### Motivation Working on the [Convert text-wrap CSS property to a shorthand](mdn/content#32341) Issue ### Related issues and pull requests - [Content PR](mdn/content#32728) - [BCD PR](mdn/browser-compat-data#22621) - Firefox Release note PR - coming soon - [Interactive examples PR](mdn/interactive-examples#2746) --------- Co-authored-by: Vadim Makeev <hi@pepelsbey.dev>
Description
Added interactive examples for:
Motivation
Working on the Convert text-wrap CSS property to a shorthand Issue
Related issues and pull requests