csswg-drafts icon indicating copy to clipboard operation
csswg-drafts copied to clipboard

[css-flexbox-1][css-position-3] static position of abspos flex children

Open fantasai opened this issue 4 years ago • 23 comments

@dholbert pointed out in https://github.com/w3c/csswg-drafts/issues/1432#issuecomment-745549275 https://github.com/w3c/csswg-drafts/issues/1432#issuecomment-746947500 and https://bugzilla.mozilla.org/show_bug.cgi?id=1682641#c9 that the newer CSS Positioned Layout model for handling the alignment properties and the flexbox spec conflict on a few points.

First, overall:

  • css-position defines abspos layout as creating an inset-modified containing block (where some of the insets are auto insets derived from the static position), and then applying alignment within that rectangle. It defines the auto offsets for a flex child to coincide with the content edges of its container.
  • css-flexbox sets the position of an abspos child more directly in https://drafts.csswg.org/css-flexbox/#abspos-items in normative prose by defining it as the position the item would have had as the sole flex item (but says something different, matching css-position, in the note in the same section).

A few places where this matters that @dholbert pointed out:

  • Per css-position, in a column flex container, the align-self property will apply in the block axis, not the main axis. But per css-flexbox, it will affect the static position in the main axis.
  • Per css-position, the justify-self property will affect the static position in the inline axis. Per css-flexbox, it can have no effect.
  • Per css-position, justify-content and align-content on the container have no effect on the child's static position. But per css-flexbox, they do.

We need to resolve these conflicts somehow.

fantasai avatar Jan 07 '21 22:01 fantasai

Our preferred resolution is just to say that Position is correct, and Flexbox was written while we were still figuring things out. We suspect the compat impact of making the changes is minimal, especially since cross-browser behavior seems all over place.

tabatkins avatar Jan 07 '21 22:01 tabatkins

Mm, actually we have a fair amount of interop here. Test results for a statically-positioned abspos child of a flex container:

  • Neither Firefox nor Chrome applies justify-self to an abspos flex child.
  • Both Firefox and Chrome apply align-self to an abspos flex child, in the cross axis.
  • Both Firefox and Chrome apply justify-content on the container to the abspos, in the main axis.
  • Firefox applies align-content on the container to the abspos, but Chrome doesn't. (Note that align-content only has an effect if the flexbox is multiline.)

Note that these behaviors only apply to abspos that are statically-positioned.

fantasai avatar Jan 07 '21 22:01 fantasai

Yeah, so the compat situation here (per fantasai's "test results" comment) seems to be that Firefox, Chromium, WebKit, and pre-Chromium Edge all directly implement the Flexbox normative prose (determining the static position as if the abspos thing were the flex container's sole flex item). The only deviation from that is that Chromium, WebKit & pre-Chromium Edge neglect to honor align-content when determining that position (which per the prose, they should honor iff the flex container is multi-line).

So browsers are mostly consistent on this right now, and have been for at least 3-4 years, which makes changes feel pretty risky from a webcompat perspective.

dholbert avatar Jan 08 '21 00:01 dholbert

OK, so proposal: define the static position rectangle of a flex child as:

  • In the main axis, the start and end margin edges of the child if it were the sole flex item in the container. [this will lock us to the position defined by justify-content on the flex container]
  • In the cross axis, we have a couple options:
    1. Use the start/end margin edges of the hypothetical flex item honoring both align-self and align-content per Firefox.
    2. Use the start/end margin edges of the hypothetical flex item pretending flex container is single-line OR use the content-box edges of the container and define that statically-positioned abspos flex children specially map their align/justify-self axes per the container's flex flow instead of the abspos containing block writing mode. (Same effect, different definition methods.)
    3. Use the content-box edges of the container, allow alignment properties to work within resulting range as for other abspos.

The first option is consistent with Firefox and the Flexbox spec. The second option is consistent with Chrome and WebKit. The third option is consistent with Chrome and WebKit in the case of row flex containers, but differs in column flex containers in whether align-self or justify-self controls the alignment. However, it means align/justify-self axis mapping would be consistent across all abspos, instead of making a special exception for statically-positioned flex child abspos.

I think if we can do the third option, in the long run (once alignment is implemented for all abspos), it will be better for authors. But we need to consider the compat impact, and would need data for that.

CC @bfgeek @cbiesinger

fantasai avatar Jan 08 '21 22:01 fantasai

Related issue: https://github.com/w3c/csswg-drafts/issues/5061

fantasai avatar Jan 16 '21 03:01 fantasai

So the question we need an answer to is, would swapping the axes of align-self vs justify-self on an abspos child of a column flex container (i.e. making align/justify match the block/inline split of other abspos, rather than the cross/main of other flex items) break content.

fantasai avatar Jun 09 '21 22:06 fantasai

Maybe @davidsgrogan can help with adding a use counter for align-self/justify-self on abspos items in a column flexbox? I guess ideally limiting to items which don't have the corresponding left/top/right/bottom property specified...

cbiesinger avatar Jun 09 '21 22:06 cbiesinger

I'm interested in this issue because it's a large chunk of failures in https://wpt.fyi/compat2021?feature=css-flexbox and am trying to understand, with the help of @svillar. I'm wondering about these two comments:

@tabatkins said:

cross-browser behavior seems all over place

@dholbert said:

browsers are mostly consistent on this right now, and have been for at least 3-4 years

Getting clarity on this would be great, and seems like it's what should determine the outcome here. If there is already a high degree of interop here, then unless the behavior creates problems for web developers, just sticking with it seems best. But there have been cases in the past when moving away from interop has been the only way to get to a sensible state, so it's not a hard rule. What should happen here?

foolip avatar Jun 10 '21 09:06 foolip

@foolip See https://github.com/w3c/csswg-drafts/issues/5843#issuecomment-757021088

fantasai avatar Jun 10 '21 17:06 fantasai

OK, so interpreting that, there's not interop currently, Firefox has one behavior and Chrome+Safari have a largely matching but different behavior. This matches the test results here: https://wpt.fyi/results/css/css-flexbox/abspos?run_id=5121058602483712&run_id=5638952738357248&run_id=5721849096830976

Since someone will have to change to align, understanding the compat risk indeed sounds important. @bfgeek @cbiesinger do you know if there are use counters for this already?

foolip avatar Jun 11 '21 10:06 foolip

@foolip, apologies, I agreed to add use counters in another thread and didn't update here.

davidsgrogan avatar Jun 11 '21 15:06 davidsgrogan

No worries, I'm just stalking this thread and don't really understand the background and context here. If use counters are being added, it seems like there's a clear next step at least: see if they're <0.01% or so and if we can figure out what kind of content triggers it.

foolip avatar Jun 11 '21 16:06 foolip

Question about proposed new behavior for which I'm gathering compat data.

  • css-flex tells engines to honor align-items when determining static position of abspos flex children
  • css-align-3 dictates ignoring align-items because the abspos children don't participate in this box's formatting context. (Right?)

Which behavior is being proposed here? E.g. These are rendered the same today. Are they rendered the same in this proposal?

<div class=column-flexbox>
  <div class=abspos-flex-child style="align-self: center;"></div>
</div>
<div class=column-flexbox style="align-items: center">
  <div class=abspos-flex-child></div>
</div>

fiddle version

Both FF and Chrome honor align-items today as can be seen in bottom flex box at the fiddle above.

davidsgrogan avatar Jul 08 '21 22:07 davidsgrogan

@davidsgrogan Hm, that's a separate question I think.

So I guess we have two questions:

  • For an abspos child of a flex container, would causing align-self and justify-self to swap axes (so they match the block/inline axis, like for abspos anywhere else) cause any breaking changes? This requires:

    • The flex container in question is a column flex container.
    • The align-self and justify-self map to different values in an absolute sense.
    • The abspos child has both insets in an axis set to auto (and is therefore statically positioned).
    • The static positioning rectangle is not equal in size to the abspos item’s margin box in that axis (thus the alignment properties can have an effect in this axis).
  • For an abspos child of any flex container, would disconnecting align-self from the parent’s align-items cause any breaking changes? This requires:

    • The abspos item is statically positioned in an axis (both insets = auto).
    • The static positioning rectangle is not equal in size to the abspos item’s margin box in that axis.
    • The align-self value is auto.
    • The parent’s align-items value is distinguishable from normal.

So I think those are the cases we need to count?

Note that if we don't disconnect align-items from abspos children, then having align-self take effect on non-statically-positioned abspos items (as specced in css-align-3) would be more likely to be a breaking change, because it would cause all of those absposes to start responding to the align-items value of their flex container parents.

fantasai avatar Jul 15 '21 20:07 fantasai

@davidsgrogan Is it possible to get counters looking at the above two sets of conditions?

tabatkins avatar Sep 07 '21 23:09 tabatkins

Yes. I have a local WIP patch. It is not as far along as https://github.com/w3c/csswg-drafts/issues/3052 though.

davidsgrogan avatar Sep 09 '21 22:09 davidsgrogan

Just update. I haven't worked on this since August but still intend to finish.

davidsgrogan avatar Nov 05 '21 19:11 davidsgrogan

@fantasai, question about this case:

<div style="display: flex; flex-flow: column; justify-content: end; width: 100px; height: 100px;">
  <div style="position: absolute; justify-self: end; align-self: start; inset-inline: 5px; width: 50px; height: 50px; "></div>
</div>

According to the first set of conditions (the "swap axis" conditions), this counts as a change in behavior, but I don't think this case would actually change under the proposal.

Applying the "swap axis" conditions to this case:

  • The flex container in question is a column flex container.

Check.

  • The align-self and justify-self map to different values in an absolute sense.

start and end are different. Check.

  • The abspos child has both insets in an axis set to auto (and is therefore statically positioned).

Yep, block axis insets are auto. Check.

  • The static positioning rectangle is not equal in size to the abspos item’s margin box in that axis (thus the alignment properties can have an effect in this axis).

50x50 and 100x100 are different. Check.

I think check 3 and check 4 need to be scoped to the cross axis, not an axis. Because the proposal only changes behavior for cross-axis positioning, I think, because this part of the proposal matches what engines already do today, right?

In the main axis, the start and end margin edges of the child if it were the sole flex item in the container. [this will lock us to the position defined by justify-content on the flex container]

davidsgrogan avatar May 08 '22 02:05 davidsgrogan

David and I chatted quickly about this case today. @fantasai am I correct in assuming that your preferred behaviour here is:

  • The static-pos rectangle for an abspos is the flexbox's content-box.
  • (and align-self / justify-self apply as described in css-align-3).
  • Then I'm confused about the desired behaviour for align-content and justify-content is it to:
    • Ignore align-content / justify-content for abspos (as the static-position rect can't "shift" for a flexbox).
    • Or to treat justify-conent similar to how flexbox is doing it today, but applied like align-self/justify-self in css-align-3? (ignoring align-content).

Ian

bfgeek avatar May 24 '22 23:05 bfgeek

The graph for the first issue from https://github.com/w3c/csswg-drafts/issues/5843#issuecomment-880999145 is at

https://chromestatus.com/metrics/feature/timeline/popularity/4237.

It overcounts because we don't have the machinery to check this condition:

The static positioning rectangle is not equal in size to the abspos item’s margin box in that axis (thus the alignment properties can have an effect in this axis).

So, even if the rectangle and margin box are equal, we count it as a behavior change.

That graph also implements the slightly different set of conditions from https://github.com/w3c/csswg-drafts/issues/5843#issuecomment-1120333927, namely only checking the cross axis, not either axis.

For the second issue in https://github.com/w3c/csswg-drafts/issues/5843#issuecomment-880999145 (disconnecting align-self):

I didn't add a UseCounter because I don't really understand it. I don't think my example that prompted it (https://github.com/w3c/csswg-drafts/issues/5843#issuecomment-876794501) is a good one. If you give an example that demonstrates the difference between current behavior and proposed behavior, and we still want a UseCounter for it, I can add one later.

Also, I'm confused by this, from the bottom of https://github.com/w3c/csswg-drafts/issues/5843#issuecomment-880999145

[not disconnecting] would cause all of those absposes to start responding to the align-items value of their flex container parents.

Absposes currently respond to the align-items value of their flex container parents, so I don't get what you mean by "start".

davidsgrogan avatar Jun 01 '22 18:06 davidsgrogan

It would be great if this could be discussed at the next meeting. The issue @davidsgrogan linked (and this example in particular) is an instance of where the flexbox spec's requirements for align-content, justify-content, align-self, and align-items can cause surprising results.

No engine currently implements align-content: stretch or align-content: normal as written in css-flexbox so a resolution to the inconsistencies between the flexbox and positioning modules could help with browser interop as well.

rreno avatar Aug 11 '22 00:08 rreno

After sitting with this for awhile it looks like the issue with align-content isn't due to disagreement between [css-flexbox-1] and [css-position-3] since position defers to flexbox for this case: https://drafts.csswg.org/css-align-3/#distribution-flex

I filed #7596 to discuss resolving the inconsistency between browser implementations and the spec for align-content.

rreno avatar Aug 11 '22 18:08 rreno

@bfgeek The ideal behavior would be to ignore align-content and justify-content, yes. Just like in Grid. That would allow the align-self and justify-self to consistently control alignment on absolutely-positioned descendants attached to the Flex/Grid container, and to allow the -content properties to focus on managing in-flow content.

(We might not be able to make that happen for Flexbox due to compat.)

fantasai avatar Aug 24 '22 19:08 fantasai