LGTM2 On Thu, Aug 19, 2021 at 9:33 PM Alex Russell <[email protected]> wrote:
> LGTM1 > > On Wednesday, August 18, 2021 at 2:30:39 PM UTC-7 Christian Biesinger > wrote: > >> Some more pieces of information: >> >> - The old version of the spec has the initial value of >> contain-intrinsic-size be `auto`; the new version changes it to `none` >> because auto now means something else. This patch keeps using `auto` >> as in the old version. >> >> - This is implemented in https://crrev.com/c/3093047 >> >> - Chromestatus did not add the link to the initial public proposal in >> the email, which expands some more on the use cases: >> https://github.com/w3c/csswg-drafts/issues/5432#issue-679230597 >> >> >> >> Christian >> >> On Thu, Aug 12, 2021 at 11:06 PM Christian Biesinger >> <[email protected]> wrote: >> > >> > Contact emails >> > >> > [email protected] >> > >> > Explainer >> > >> > None >> > >> > Specification >> > >> > https://www.w3.org/TR/css-sizing-4/#intrinsic-size-override >> > >> > Summary >> > >> > This converts contain-intrinsic-size to a shorthand and adds >> > corresponding contain-intrinsic-{width,height} properties as well as >> > contain-intrinsic-{inline,block} size. For example, this will make the >> > following work: <div style="writing-mode: vertical-lr; contain: size; >> > contain-intrinsic-inline-size: 100px; contain-intrinsic-block-size: >> > 50px;"></div> >> > >> > >> > >> > Blink component >> > >> > Blink>Layout >> > >> > Search tags >> > >> > contain-intrinsic-size, contain intrinsic size, logical >> > >> > TAG review >> > >> > https://github.com/w3ctag/design-reviews/issues/624 >> > >> > TAG review status >> > >> > Pending >> > >> > Risks >> > >> > >> > >> > Interoperability and Compatibility >> > >> > >> > >> > Gecko: Positive ( >> https://github.com/mozilla/standards-positions/issues/512) >> > >> > WebKit: No signal >> > (https://lists.webkit.org/pipermail/webkit-dev/2021-April/031787.html) >> > >> > Web developers: No signals >> > >> > >> > Debuggability >> > >> > like any other CSS property >> > >> > >> > >> > Is this feature fully tested by web-platform-tests? >> > >> > Yes >> > >> > Flag name >> > >> > >> > >> > Requires code in //chrome? >> > >> > False >> > >> > Tracking bug >> > >> > https://crbug.com/1157844 >> > >> > Estimated milestones >> > >> > >> > >> > Link to entry on the Chrome Platform Status >> > >> > https://chromestatus.com/feature/5709654999957504 >> > >> > Motivation >> > >> > For the common case of using c-i-s to provide a better scrollbar >> > approximation for content-visibility: auto, being able to just specify >> > an intrinsic height is helpful. And of course the logical versions >> > make this easier for vertical writing mode pages. >> > >> > This also paves the way for contain-intrinsic-size: auto and will make >> > 1d size containment work more straightforwardly and thus help with >> > container queries. >> > >> > This intent message was generated by Chrome Platform Status. >> > -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/90041ace-58b2-461e-a2b2-9b6c9e9fae4an%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/90041ace-58b2-461e-a2b2-9b6c9e9fae4an%40chromium.org?utm_medium=email&utm_source=footer> > . > -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfWyoxYagNVgN9amodOE2ub0s9UmzuqjSLcgvPu%2BALt9tA%40mail.gmail.com.
