Skip to content

Conversation

@frivoal
Copy link
Collaborator

@frivoal frivoal commented Aug 25, 2023

Recast word-boundary-detection: auto() as word-break: auto-phrase, as well as word-boundary-detection: manual as word-break: manual.

This allows retiring the word-boundary-detection property.

Also, the word-boundary-expansion property needed to gain its own auto-phrase value, since it could no longer depend on word-boundary-detection: auto().

Since there's no longer a pair of word-boundary-* properties, there's no need to keep the two names related, and the original name was quite a mouthful. word-space-transform is simpler, and better show-cases the similarities with text-transform.

Supporting sections had to be reorganized as well to keep the whole thing coherent.

This addresses #7193 (and w3c/i18n-activity#1454). It builds upon and is meant to replace #9216 (which itself builds upon and replaces #8974).

(co-authored with @fantasai)


Note: the <wpt> annotations have not been fully updated yet. Ignore them for now: I'll update the tests and the corresponding <wpt> annotations in the spec once we agree on the behavior described in this PR.

@frivoal
Copy link
Collaborator Author

frivoal commented Aug 25, 2023

This PR moves quite a bit of text around, so it's not too easy too review as a source diff. Here are:

@frivoal frivoal changed the title [css-text-3] auto-phrase [css-text-4] auto-phrase Aug 25, 2023
Copy link
Contributor

@kojiishi kojiishi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

frivoal and others added 2 commits August 30, 2023 15:29
Recast word-boundary-detection:auto() as word-break:auto-phrase,
as well as word-boundary-detection:manual as word-break:manual.

This allows retiring the word-boundary-detection property.

Also, the word-boundary-expansion property needed to gain its own
auto-phrase value, since it could no longer depend on
word-boundary-detection:auto().

Supporting sections had to be reorganized as well to keep the whole
thing coherent.

See w3c#7193.

Co-authored-by: fantasai <fantasai.bugs@inkedblade.net>
Since there's no longer a pair of word-boundary-* properties, there's no
need to keep the two names related, and the original name was quite a
mouthful. 'word-space-transform' is simpler, and better show-cases the
similarities with text-transform.

Part of w3c#7193

Co-authored-by: fantasai <fantasai.bugs@inkedblade.net>
frivoal added a commit to frivoal/wpt that referenced this pull request Aug 30, 2023
frivoal added a commit to frivoal/wpt that referenced this pull request Aug 30, 2023
Adjust word-boundary-expansion tests to become word-space-transform
tests, renaming them and accounting for the slight behavior difference
(timing of the transformation moved from before phase 1 to between phase
1 and 2).
frivoal added a commit to frivoal/wpt that referenced this pull request Aug 30, 2023
frivoal added a commit to frivoal/wpt that referenced this pull request Aug 30, 2023
Adjust word-boundary-expansion tests to become word-space-transform
tests, renaming them and accounting for the slight behavior difference
(timing of the transformation moved from before phase 1 to between phase
1 and 2).
@kojiishi kojiishi merged commit c9f57a9 into w3c:main Aug 31, 2023
@frivoal frivoal deleted the phrases branch September 1, 2023 08:38
@frivoal frivoal removed the Agenda+ label Sep 1, 2023
frivoal added a commit to frivoal/wpt that referenced this pull request Sep 1, 2023
word-break and word-space-transform: auto-phrase tests

(WIP)
frivoal added a commit to frivoal/wpt that referenced this pull request Sep 1, 2023
word-break and word-space-transform: auto-phrase tests
frivoal added a commit to frivoal/wpt that referenced this pull request Sep 1, 2023
word-break and word-space-transform: auto-phrase tests
frivoal added a commit to frivoal/wpt that referenced this pull request Sep 1, 2023
frivoal added a commit to web-platform-tests/wpt that referenced this pull request Sep 1, 2023
frivoal added a commit to web-platform-tests/wpt that referenced this pull request Sep 1, 2023
Adjust word-boundary-expansion tests to become word-space-transform
tests, renaming them and accounting for the slight behavior difference
(timing of the transformation moved from before phase 1 to between phase
1 and 2).
frivoal added a commit to web-platform-tests/wpt that referenced this pull request Sep 1, 2023
word-break and word-space-transform: auto-phrase tests
frivoal added a commit to web-platform-tests/wpt that referenced this pull request Sep 1, 2023
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Sep 13, 2023
…-drafts#9242, a=testonly

Automatic update from web-platform-tests
Update css-text tests to match w3c/csswg-drafts#9242

syntax tests

--
Update css-text tests to match w3c/csswg-drafts#9242

Adjust word-boundary-expansion tests to become word-space-transform
tests, renaming them and accounting for the slight behavior difference
(timing of the transformation moved from before phase 1 to between phase
1 and 2).

--
Update css-text tests to match w3c/csswg-drafts#9242

word-break and word-space-transform: auto-phrase tests

--
Extra css-text tests to match w3c/csswg-drafts#9242

--
Move tests in reference subfolder

--

wpt-commits: 9a8dd9aae7bd12896f91bfe1f8bb8111bb00d24d, 40f20b3118ff6a1511e897f89f67a5a850dfe743, 4ed7098104db0a041d7b6ed7dcd4aa6709dec2f3, a91411b722c19700b70e0e22da0cc085420961a3, 63850c62203d786cd7c01d159b5a4ca7be950bb5
wpt-pr: 41712
vinnydiehl pushed a commit to vinnydiehl/mozilla-unified that referenced this pull request Sep 14, 2023
…-drafts#9242, a=testonly

Automatic update from web-platform-tests
Update css-text tests to match w3c/csswg-drafts#9242

syntax tests

--
Update css-text tests to match w3c/csswg-drafts#9242

Adjust word-boundary-expansion tests to become word-space-transform
tests, renaming them and accounting for the slight behavior difference
(timing of the transformation moved from before phase 1 to between phase
1 and 2).

--
Update css-text tests to match w3c/csswg-drafts#9242

word-break and word-space-transform: auto-phrase tests

--
Extra css-text tests to match w3c/csswg-drafts#9242

--
Move tests in reference subfolder

--

wpt-commits: 9a8dd9aae7bd12896f91bfe1f8bb8111bb00d24d, 40f20b3118ff6a1511e897f89f67a5a850dfe743, 4ed7098104db0a041d7b6ed7dcd4aa6709dec2f3, a91411b722c19700b70e0e22da0cc085420961a3, 63850c62203d786cd7c01d159b5a4ca7be950bb5
wpt-pr: 41712
Lightning00Blade pushed a commit to Lightning00Blade/wpt that referenced this pull request Dec 11, 2023
Lightning00Blade pushed a commit to Lightning00Blade/wpt that referenced this pull request Dec 11, 2023
Adjust word-boundary-expansion tests to become word-space-transform
tests, renaming them and accounting for the slight behavior difference
(timing of the transformation moved from before phase 1 to between phase
1 and 2).
Lightning00Blade pushed a commit to Lightning00Blade/wpt that referenced this pull request Dec 11, 2023
word-break and word-space-transform: auto-phrase tests
Lightning00Blade pushed a commit to Lightning00Blade/wpt that referenced this pull request Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants