Skip to content
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 word-wrap: anywhere support #912

Merged
merged 4 commits into from
Oct 2, 2024
Merged

Conversation

nspector
Copy link
Contributor

@nspector nspector commented Oct 2, 2024

I confirm that this contribution is made under a BSD license and that I have the authority necessary to make this contribution on behalf of its copyright owner.


Description

Adds support for word-wrap: anywhere - https://developer.mozilla.org/en-US/docs/Web/CSS/overflow-wrap

Motivation and Context

Adds support for a supported value of word-wrap

How Has This Been Tested?

Snapshots have been updated.

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

Review the following points, and put an x in all the boxes that apply. If you’re unsure about any of these, don’t hesitate to ask. We’re here to help!

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

Copy link

changeset-bot bot commented Oct 2, 2024

⚠️ No Changeset found

Latest commit: 25973df

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Oct 2, 2024

Someone is attempting to deploy a commit to the Atomizer Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Oct 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
acss-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 4:47pm

@@ -4430,6 +4430,7 @@ module.exports = [
{
bw: 'break-word',
n: 'normal',
a: 'anywhere',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To prevent overflow, an otherwise unbreakable string of characters — like a long word or URL — may be broken at any point if there are no otherwise-acceptable break points in the line. No hyphenation character is inserted at the break point. Soft wrap opportunities introduced by the word break are considered when calculating min-content intrinsic sizes.

snyamathi
snyamathi previously approved these changes Oct 2, 2024
redonkulus
redonkulus previously approved these changes Oct 2, 2024
@redonkulus redonkulus merged commit 678b6ca into acss-io:main Oct 2, 2024
2 of 3 checks passed
@nspector nspector deleted the nspector/wordWrap branch October 2, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants