-
Notifications
You must be signed in to change notification settings - Fork 715
[css-grid] Non-existent line names in abspos grid items #3445
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
Comments
I think it makes sense to clarify the spec, but also that the Firefox/Edge behavior is correct. We end up spanning to the 'auto' line for non-existent lines only because abspos can't add lines. Either way it's a strange error case. :) Agenda+ in case anyone has a different opinion. |
…ames to aspos section about non-existent lines. #3445
I think the "and considered to have foo as name" is a bit misleading. There is no such thing as an implicit line that only matches a single name. Since that implicit line exist it also matches any other name, so it also matches "bar". Given that an implicit line does exist in this case, and that implicit lines match all names, the spec text from 9.1 that you quote does not apply since the line is neither "non-existent" nor outside the existing implicit grid. |
BTW, I agree with Firefox/Edge behavior, I just wanted the note on the spec to make things clearer (at least for me). |
The CSS Working Group just discussed
The full IRC log of that discussion<dael> Topic: Non-existent line names in abspos grid items<dael> github: https://github.com//issues/3445 <tantek> fantasai, no, you're cherry picking lol. Appendices A, B, C, E, F etc. are all informative in CSS 2.1 <dael> fantasai: Wanted to confirm everything with WG since it's CR <dael> Rossen: Can you summarize? <tantek> s/ E,/ D,/ <myles> s/i'd like to implement these/i'd like to implement these someday eventually/ <dael> fantasai: Issue was about the refence for line name. Currently define implicit lines have all the names. Abspos referring to line name not in explicit grid. Had been a case where someone referenced a non existant line in inflow grid and caused a line to be created and now abspos is looking for a different line name. We defined it's looking for implicit line. Added a note to clarify, everyone seems in agreement <fantasai> tantek, A says it's informative explicitly <dael> florian: Close no change or close with clarification note? <tantek> Yeah, Appendix E stacking contexts in CSS 2.1 should have been made a chapter, not an Appendix. All other appendices are informative. <dael> fantasai: Close with clarification note <lajava> I've been discussing this with rego and we agree that the note clarifies the issue <fantasai> tantek, B has to be normative because it contains the normative references <dael> Rossen: Objections to close this issue and accept the clarification note? <dael> RESOLVED: close this issue and accept the clarification note <fantasai> tantek, C says it's informative explicitly |
Imagin the following example we have a grid container with 2 rows and 2 columns (the grid container has
position: relative
so it's the containing block of the baspos descendants).A regular item is placed with
grid-column: 3 / foo; grid-row: 3 / foo;
, thus an implicit grid line (4) is created and considered to havefoo
as name.Then if we have an abspos item with
grid-column: 3 / bar; grid-row: 3 / bar;
how we should resolve thebar
line?Only checking this part of the spec it seems pretty clear that
bar
line should point to the implicit line (4) (that matches Firefox/Edge behavior, note that Blink/WebKit consider that line to beauto
for the abspos item).However reading this part I have more doubts about if it should be considered a non-existent line or not. If
bar
is not a non-existent line because we have an implicit line and we assume it could have any name, shouldn't we clarify a little bit this last sentence?This issue is related to bug #915551 reported by @MatsPalmgren on Chromium.
The text was updated successfully, but these errors were encountered: