-
Notifications
You must be signed in to change notification settings - Fork 671
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-inline-3] text-box-trim accumulation #5426
Comments
I was just looking into how to solve text-box-trim on the first/last blocks in a section or article, and having the possibility to set <section>
<h2>A title</h2>
<p>A paragraph</p>
<p>Another paragraph</p>
</section> section {
text-box-trim: both;
text-box-edge: auto;
}
h2, p {
text-box-trim: auto;
} Would |
The CSS Working Group just discussed The full IRC log of that discussion<ntim> fantasai: leading-trim can drill through elements, if you set it on the div, it will affect the paragraph in the div<ntim> fantasai: however, if the both the paragraph and div say leading-trim: trim with different metrics, then we might want to look at the closest metric <ntim> fantasai: we might something similar to page-break: auto, where we can break or not break <ntim> fantasai: we could add an auto value to text-box-trim <fantasai> where auto means, do whatever your parent says <astearns> q+ <miriam> ack dbaron <ntim> dbaron: just to restate the question: the choice between the metric from the closest ancestor, or the metric that is determine the most amount of <ntim> myles: this reminds me of text-decoration where we can never remove them <miriam> ack astearns <dbaron> s/is determine the most amount of/takes the largest amount off/ <ntim> astearns: the proposal sounds ok in theory, but I want to see the proposal fleshed out and try some examples myself before fulling signing off <ntim> astearns: this kind of sounds like trim collapsing <ntim> ...and you want to avoid it <ntim> florian: aren't we narrowing down to collapsing from the closest ancestor? <florian> s/down to collapsing/down to trimming/ <ntim> astearns: Is this discussion enough to come up with a concrete proposal to evaluate? <hober> scribe: myles <myles> florian: it seems to me that we have a clear feeling that the largest metric is not what we want. Between the other options, we didn't discuss clearly. <myles> astearns: dbaron originally asked "do you have a preference"? <astearns> s/"/so / <myles> dbaron: I don't have a preference. I don't understand what the issue is. I think it's closely tied to the syntax which I don't have in my head rn <astearns> s/"?/?/ <fantasai> The three options were: take the closest metric where metric is available, take metric from closest that asks for trimming, take metric with largets amount of trimming <fantasai> And I guess another one would be "outermost" metric <astearns> no to the third <myles> myles: how do we make progress here? <myles> fantasai: astearns can help with examples, but we'll write a draft and be more specific <myles> astearns: that sounds good <myles> fantasai: ok |
@johannesodland Yes. This is actually the currently-specced behavior, but there's no conflict resolution specified if both of them specify a text-box-trim value. :) |
I propose that we go with "take metric from closest that asks for trimming" among the three listed from the meeting notes. I don't think we need any new keyword values. More precise wording: a line is trimmed according to the Note: we don't need this spec language for Note: This is different than inheritance, because it only applies to the first line of any descendant of the ancestor, not to all first lines of different paragraphs. The line referred to by |
Taking the innermost metric probably is pretty easy to understand. I'm just worried we'll have a problem like we do with collapsed table borders, where the obvious thing was to have the innermost (“most specific“) win, but it actually works out to not do the right thing on most tables. :] (The example is, btw: same-width solid borders applied to the table, row group, and cells, with the cells being gray and the table/row group being black. Similar to this. You end up with all-gray borders, which isn't want you want: it's actually the outer elements that you want to win here, because the borders that coincide with them are the “most special”.) |
My suggestion is to resolve on this behavior in the spec because it's easy enough to understand, prototype it in a browser engine, and then ask developers to try it and gather their feedback. If they find problems we can reconsider the semantics. |
I think that while tentatively implementing "the right thing" even if we're not sure if it's going to be web-compatible and reverting if it isn't can often be a good idea, shipping a thing which may not be the best approach and changing later it if people don't like it is a lot dicier, as we may be trapped by compat by then, even if we do conclude that the alternative behavior would have been better. Now, if it's about implementing without shipping in order to let people experiment with prototypes, sure. But then I don't think it needs to be in the spec. |
The CSS Working Group just discussed
The full IRC log of that discussion<Frances> fantasai: consider an example with a div where inside there is a p, both asking for text-box:trim. Shared by two elements nested inside each other, which trimming should we perform? Such as different metrics?<Rossen_> q <Rossen_> ack florian <Frances> fantasai: Can choose outer most or inner most or most constrictive one. Inner most is most specific, the alternative would be outermost to match up with something outside. <Frances> fantasai: Which one would be the most appropriate metric? <Frances> florian: If a sense of which one is right we can choose it, or we can implement the use case driven one. <Frances> rossen: if strong use cases come, would be much easier to select. <dbaron> innermost seems reasonable, at least <Frances> PROPOSAL: Choosing the innermost for textbox:trim for most requested trim metric. <Frances> RESOLVED: Choosing the innermost for textbox:trim for most requested trim metric. |
This patch adds a test for when the element to trim has different `text-box-edge` value from the `text-box-edge` value of the element that has `text-box-trim`. Discussed and resolved at: w3c/csswg-drafts#5426 Bug: 40254880 Change-Id: I5685fbaebddb8c7429b3b041c5fe0f88b982fc3d
This patch adds a test for when the element to trim has different `text-box-edge` value from the `text-box-edge` value of the element that has `text-box-trim`. Discussed and resolved at: w3c/csswg-drafts#5426 Bug: 40254880 Change-Id: I5685fbaebddb8c7429b3b041c5fe0f88b982fc3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5547563 Auto-Submit: Koji Ishii <[email protected]> Reviewed-by: Kent Tamura <[email protected]> Commit-Queue: Kent Tamura <[email protected]> Cr-Commit-Position: refs/heads/main@{#1303166}
This patch adds a test for when the element to trim has different `text-box-edge` value from the `text-box-edge` value of the element that has `text-box-trim`. Discussed and resolved at: w3c/csswg-drafts#5426 Bug: 40254880 Change-Id: I5685fbaebddb8c7429b3b041c5fe0f88b982fc3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5547563 Auto-Submit: Koji Ishii <[email protected]> Reviewed-by: Kent Tamura <[email protected]> Commit-Queue: Kent Tamura <[email protected]> Cr-Commit-Position: refs/heads/main@{#1303166}
This patch adds a test for when the element to trim has different `text-box-edge` value from the `text-box-edge` value of the element that has `text-box-trim`. Discussed and resolved at: w3c/csswg-drafts#5426 Bug: 40254880 Change-Id: I5685fbaebddb8c7429b3b041c5fe0f88b982fc3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5547563 Auto-Submit: Koji Ishii <[email protected]> Reviewed-by: Kent Tamura <[email protected]> Commit-Queue: Kent Tamura <[email protected]> Cr-Commit-Position: refs/heads/main@{#1303166}
I am not sure if I understood the discussion result correctly. When nesting different elements with different line-heights, the height of the largest line-height is used for the entire group. For me as developer it would be most intuitive if text-box-trim would behave in a similar way. Here are two examples and in both examples the largest font-size should be used for the trim calculation:
For the same font size couldn't we do just trim the minimum? for example |
Thank you @jantimon for the great feedback. Let me explain my understanding of the resolution. The #8829 resolved to split the line-box-affecting aspects of <div style="text-box-trim: both; text-box-edge: text">
<div style="text-box-edge: leading">
<div style="text-box-edge: cap text">FIRST</div>
<div>MIDDLE</div>
<div style="text-box-edge: text alphabetic">LAST</div>
</div>
</div> Since the resolution is "innermost", the top of the "FIRST" should be trimmed at the The split in #8829 made applying When we start discussing @fantasai Please correct me if any. |
thanks @kojiishi for your quick response - so it is not in that case "innermost" makes perfect sense for many different scenarios e.g. optical vertical text centering: |
…stonly Automatic update from web-platform-tests [text-box-trim] Test accumulations This patch adds a test for when the element to trim has different `text-box-edge` value from the `text-box-edge` value of the element that has `text-box-trim`. Discussed and resolved at: w3c/csswg-drafts#5426 Bug: 40254880 Change-Id: I5685fbaebddb8c7429b3b041c5fe0f88b982fc3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5547563 Auto-Submit: Koji Ishii <[email protected]> Reviewed-by: Kent Tamura <[email protected]> Commit-Queue: Kent Tamura <[email protected]> Cr-Commit-Position: refs/heads/main@{#1303166} -- wpt-commits: 5e6793faa62943090d88eca7fa6390219f06832d wpt-pr: 46340
…stonly Automatic update from web-platform-tests [text-box-trim] Test accumulations This patch adds a test for when the element to trim has different `text-box-edge` value from the `text-box-edge` value of the element that has `text-box-trim`. Discussed and resolved at: w3c/csswg-drafts#5426 Bug: 40254880 Change-Id: I5685fbaebddb8c7429b3b041c5fe0f88b982fc3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5547563 Auto-Submit: Koji Ishii <[email protected]> Reviewed-by: Kent Tamura <[email protected]> Commit-Queue: Kent Tamura <[email protected]> Cr-Commit-Position: refs/heads/main@{#1303166} -- wpt-commits: 5e6793faa62943090d88eca7fa6390219f06832d wpt-pr: 46340
We allow ancestors to affect leading trimming on descendants, so that common cases like
will work for authors. But we don't specify what happens if various ancestors have different values for
leading-trim
.Maybe we should do something similar to
break-before
and switch to an auto/no/yes model where auto does whatever the ancestor says and otherwise the innermost value wins?The text was updated successfully, but these errors were encountered: