-
Notifications
You must be signed in to change notification settings - Fork 166
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
Add text-wrap: pretty to _base_typography.scss #5090
Conversation
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.
Small suggestion on the changelog notes, but otherwise looks good.
This is the latest build that includes Chrome, so we can see changes it detected. Most of them are exactly what we expect, although there are few cases where it does a bit unexpected things. It wraps the date time numbers as if they were words: Sometimes adjusted wrapping affect not only the last line: But there is likely no way to prevent any of these. |
@bartaz What are your thoughts on those two cases? Do they concern you? I'm not sure I understand the second example you linked - that appears as expected behavior, to me. Can you explain? |
Co-authored-by: Bartek Szopka <83575+bartaz@users.noreply.github.com>
I think the date issue is the riskier one. The wrapping is not a major concern (since we've already embarked on simplifying things and sacrificing some control in the process). It should be fairly easy to recover from the date issue, e.g. with a |
Done
Added
text-wrap: pretty
to _base_typography.scss in order to prevent text widows on browsers that support thepretty
value.Open demo
QA: Examine text elements across documentation site and see that no text widows should exist
Check if PR is ready for release
If this PR contains Vanilla SCSS code changes, it should contain the following changes to make sure it's ready for the release:
Feature 🎁
,Breaking Change 💣
,Bug 🐛
,Documentation 📝
,Maintenance 🔨
.package.json
should be updated relative to the most recent release, following semver convention:Screenshots
See below comments for any edge case scenario screenshots.