WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
285257
text-box-trim accumulation fails when updating the CSS dynamically
https://bugs.webkit.org/show_bug.cgi?id=285257
Summary
text-box-trim accumulation fails when updating the CSS dynamically
mic.gallego
Reported
2024-12-30 22:30:19 PST
Created
attachment 473720
[details]
video Hi, According to this issue (
https://github.com/w3c/csswg-drafts/issues/5426
) when the text-box: trim-both is set on a parent, it should trim start on the first element, and trim end on the last element. This is what I am experiencing on Safari (both stable and nightly) but only on initial load. If I update the CSS in the inspector, then the output end up incorrect, with the second element having it's start being trimmed. Please see the attach video (on initial load it is correct, but incorrect when you update it).
Attachments
video
(13.79 MB, video/quicktime)
2024-12-30 22:30 PST
,
mic.gallego
no flags
Details
Patch
(20.04 KB, patch)
2025-01-05 13:04 PST
,
zalan
no flags
Details
Formatted Diff
Diff
[fast-cq]Patch
(19.91 KB, patch)
2025-01-06 07:48 PST
,
zalan
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2025-01-05 12:57:48 PST
<
rdar://problem/142386761
>
zalan
Comment 2
2025-01-05 13:03:51 PST
Thank you for filing this bug report on text-box!
zalan
Comment 3
2025-01-05 13:04:55 PST
Created
attachment 473782
[details]
Patch
Tim Nguyen (:ntim)
Comment 4
2025-01-05 16:24:28 PST
Comment on
attachment 473782
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=473782&action=review
> LayoutTests/imported/w3c/web-platform-tests/css/css-inline/text-box-trim/text-box-trim-set-dynamically-expected.html:4 > +<link rel="match" href="text-box-trim-set-dynamically-ref.html">
ditto
> LayoutTests/imported/w3c/web-platform-tests/css/css-inline/text-box-trim/text-box-trim-set-dynamically-ref.html:4 > +<link rel="match" href="text-box-trim-set-dynamically-ref.html">
You can remove the link[rel=match] from the ref/expected files.
zalan
Comment 5
2025-01-06 07:48:40 PST
Created
attachment 473808
[details]
[fast-cq]Patch
EWS
Comment 6
2025-01-06 11:57:07 PST
Committed
288480@main
(bbbde60e23e9): <
https://commits.webkit.org/288480@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 473808
[details]
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug