-
Notifications
You must be signed in to change notification settings - Fork 689
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
[css-text] Does break-spaces + break-word allow breaking before the first trailing white-space ? #3221
Comments
Correct.
You're right. break-spaces-003.html was written back when the spec said something different, and is now incorrect. I'll make a pull request to delete it. |
This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221
This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221
The tests are fixed, so closing, and this wasn't really a spec problem, so closing as OutOfScope, for the lack of a more approprioate status. |
Automatic update from web-platform-tests[css-text] Remove stale tests This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221 -- Merge pull request #13589 from frivoal/text-3221 [css-text] Remove stale test -- wpt-commits: 267f7dcb3019ca3a4483bc81de7270e37e0d7421, cabd743ac65d556c29f179952bde80d162768914 wpt-pr: 13589
Automatic update from web-platform-tests[css-text] Remove stale tests This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221 -- Merge pull request #13589 from frivoal/text-3221 [css-text] Remove stale test -- wpt-commits: 267f7dcb3019ca3a4483bc81de7270e37e0d7421, cabd743ac65d556c29f179952bde80d162768914 wpt-pr: 13589
Automatic update from web-platform-tests[css-text] Remove stale tests This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221 -- Merge pull request #13589 from frivoal/text-3221 [css-text] Remove stale test -- wpt-commits: 267f7dcb3019ca3a4483bc81de7270e37e0d7421, cabd743ac65d556c29f179952bde80d162768914 wpt-pr: 13589 UltraBlame original commit: 7288243365f58a866a6abc313bb6e7df5d6defa2
Automatic update from web-platform-tests[css-text] Remove stale tests This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221 -- Merge pull request #13589 from frivoal/text-3221 [css-text] Remove stale test -- wpt-commits: 267f7dcb3019ca3a4483bc81de7270e37e0d7421, cabd743ac65d556c29f179952bde80d162768914 wpt-pr: 13589 UltraBlame original commit: 7288243365f58a866a6abc313bb6e7df5d6defa2
Automatic update from web-platform-tests[css-text] Remove stale tests This was written based on a previous version of the spec, and is no longer correct. css/css-text/overflow-wrap/overflow-wrap-break-word-002.html checks the correct behavior instead. Reported in w3c/csswg-drafts#3221 -- Merge pull request #13589 from frivoal/text-3221 [css-text] Remove stale test -- wpt-commits: 267f7dcb3019ca3a4483bc81de7270e37e0d7421, cabd743ac65d556c29f179952bde80d162768914 wpt-pr: 13589 UltraBlame original commit: 7288243365f58a866a6abc313bb6e7df5d6defa2
According to the spec, 'white-space: break-space' allows breaking AFTER any preserved white space.
https://drafts.csswg.org/css-text-3/#valdef-white-space-break-spaces
Additionally, the spec states the following on 'overflow-wrap: break-word':
https://drafts.csswg.org/css-text-3/#valdef-overflow-wrap-break-word
So, combining both properties with such values I understand that a there is a breaking opportunity before the first preserved white-space following a word if it overflows. However, this shouldn't be allowed if there are other breaking opportunities before (eg, a leading white-space).
If I understood it correctly, then I think that these two test are somehow contradictory:
css/css-text/white-space/break-spaces-003.html
css/css-text/overflow-wrap/overflow-wrap-break-word-002.html
The first test (break-spaces-003.html) expects the second ' XX' string to be wrapped, preserving the white-space as leading character. However, the second test expects the word to be wrapped after the leading white-space, so that 'FAIL ' goes to the second line.
The text was updated successfully, but these errors were encountered: