-
Notifications
You must be signed in to change notification settings - Fork 2k
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
added pretty to text-wrap-style #22645
Conversation
Co-authored-by: Queen Vinyl Da.i'gyu-Kazotetsu <vinyldarkscratch@gmail.com>
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.
LGTM
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.
Ah, I just noticed that this value is unsupported in all browsers. We currently aren't enforcing this specific linting rule for the CSS category, but this would fail the lint if it was.
Is this feature supported behind a flag or in any beta/nightly releases? If so, let's add that data so this would pass the linter when it's enabled. If not, let's wait to add this until a browser has support.
Currently |
Well, now this PR is empty. 😆 I'm going to go ahead and close this since no browsers support the value, but thank you anyways! |
Summary
Added
pretty
value to thetext-wrap-style
propertyTest results and supporting details
The property is not supported in Chromium browsers or safari and the value is not supported in FF 124
Related issues