(I can't count... tkent already gave LGTM1 - but you have 3 now)

On 10/12/22 2:03 PM, Chris Harrelson wrote:
LGTM2

On Wed, Oct 12, 2022 at 11:02 AM Mike Taylor <miketa...@chromium.org> wrote:

    Thanks - LGTM1.

    On 10/12/22 12:04 PM, Ian Kilpatrick wrote:
    The tests failing are due to the skew between a WPT sync, and WPT
    running against an "older" version of chrome.

    These tests pass in Chrome Canary.

    Ian

    On Wed, Oct 12, 2022 at 9:02 AM Philip Jägenstedt
    <foo...@chromium.org> wrote:

        The problem with feature detection here happens once in a
        while, and flex gap is a prominent previous example. But when
        it's already happened, there's really not much we can do to
        fix the situation. So I'm very much inclined to LGTM this.

        But I see some tests
        
<https://wpt.fyi/results/css?label=master&label=experimental&aligned&view=subtest&q=flex-align-baseline%20or%20grid-align-baseline%20or%20grid-justify-baseline>
        are failing, are those failures accounted for, with bugs
        filed? If this is an area with messy interop, it might pay
        off to cross every t and dot every i here.

        On Tue, Oct 11, 2022 at 7:30 PM Ian Kilpatrick
        <ikilpatr...@chromium.org> wrote:



            On Tue, Oct 11, 2022 at 7:22 AM Mike Taylor
            <miketa...@chromium.org> wrote:

                On 10/11/22 12:11 AM, Ian Kilpatrick wrote:


                On Mon, Oct 10, 2022 at 8:02 PM Yoav Weiss
                <yoavwe...@chromium.org> wrote:



                    On Tue, Oct 11, 2022 at 1:37 AM TAMURA, Kent
                    <tk...@chromium.org> wrote:

                        LGTM1.
                        Two other browsers already shipped this. No
                        doubt to proceed this.

                        On Sat, Oct 8, 2022 at 4:15 AM Ian
                        Kilpatrick <ikilpatr...@chromium.org> wrote:


                                    Contact emails

                            ikilpatr...@chromium.org


                                    Explainer

                            None


                                    Specification

                            https://drafts.csswg.org/css-align-3/#baseline-rules


                                    Summary

                            This feature allows developers to align
                            items within either flex or grid layout
                            by their last baseline, instead of their
                            first. This is done via the following
                            properties: align-items: last baseline;
                            /* flex/grid container */ justify-items:
                            last baseline; /* grid container */
                            align-self: last baseline; /* flex/grid
                            item */ justify-self: last baseline; /*
                            grid item */



                                    Blink component

                            Blink>Layout
                            
<https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ELayout>


                                    TAG review

                            None - I don't believe CSS Align 3 ever
                            went through Tag Review. However I'm
                            happy to file one and cc/ the
                            specification editors.


                                    TAG review status

                            Not applicable


                                    Risks



                                    Interoperability and Compatibility



                            /Gecko/: Shipped/Shipping (note -
                            relatively basic support)

                            /WebKit/: Shipped/Shipping "It's
                            complicated". WebKit currently parses
                            "align-items: last baseline" however
                            doesn't support it currently in layout
                            (for Stable versions of the browser).
                            https://caniuse.com/?search=last%20baseline
                            <https://caniuse.com/?search=last+baseline> This
                            is currently changing however with
                            recent commits by WebKit implementing
                            the "last baseline" behaviour. E.g.
                            https://bugs.webkit.org/show_bug.cgi?id=245792

                            /Web developers/: No signals

                            /Other signals/:


                                    Activation

                            This feature will likely be difficult to
                            use initially for web developers for a
                            few reasons. Support and implementation
                            status is patchy. While adding tests for
                            this feature...
                            
https://wpt.fyi/results/css?label=master&label=experimental&aligned&view=subtest&q=flex-align-baseline%20or%20grid-align-baseline%20or%20grid-justify-baseline
                            
<https://wpt.fyi/results/css?label=master&label=experimental&aligned=&view=subtest&q=flex-align-baseline+or+grid-align-baseline+or+grid-justify-baseline>
                            ...existing implementations typically
                            only supported relatively simple cases
                            well. This primarily was because various
                            specifications which this feature
                            interacts with didn't specify what the
                            last baseline of various boxes should
                            be. Additionally "@supports(align-items:
                            last baseline)" will return true for
                            many versions of browsers that don't
                            have any (or partial) layout support for
                            this feature. Web developers may rely on
                            UA versions and/or javascript to check
                            actual support.


                    This is not great.. Any ways in which we can
                    improve the story? e.g. can we push for better
                    WPT coverage? Have this feature be a part of
                    Interop 23? Rename the feature into something
                    that can be reliably detected? Something else?


                We'll likely be able to incorporate these tests into
                interop 2023 for both grid & flexbox which will
                hopefully improve interop. Renaming the feature
                isn't a great option (IMO) as it describes what it
                does well, and Firefox has already shipped support.

                Quick question on the levels of support. You mention
                that Firefox has basic support. And WebKit landed
                some support in
                https://github.com/WebKit/WebKit/pull/4799. Do you
                have a sense of how Safari's implementation (maybe
                testable in nightly builds) compares to Firefox and
                your implementation?


            The parts which begin connecting the above patch into
            flex/grid haven't reached a Safari TP which we can test
            yet. I expect initially it'll be similar to Firefox's
            level of support (handle basic cases well but complex
            interactions less well).

            Ian


                Ian




                                    WebView application risks

                            Does this intent deprecate or change
                            behavior of existing APIs, such that it
                            has potentially high risk for Android
                            WebView-based applications?



                                    Debuggability

                            Basic CSS property debugging in DevTools.



                                    Will this feature be supported
                                    on all six Blink platforms
                                    (Windows, Mac, Linux, Chrome OS,
                                    Android, and Android WebView)?

                            Yes


                                    Is this feature fully tested by
                                    web-platform-tests
                                    
<https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>?

                            Yes
                            
https://wpt.fyi/results/css?label=master&label=experimental&aligned&view=subtest&q=flex-align-baseline%20or%20grid-align-baseline%20or%20grid-justify-baseline
                            
<https://wpt.fyi/results/css?label=master&label=experimental&aligned=&view=subtest&q=flex-align-baseline+or+grid-align-baseline+or+grid-justify-baseline>


                                    Flag name

                            --enable-experimental-web-platform-features
                            or --enable-blink-features=CSSLastBaseline


                                    Requires code in //chrome?

                            False


                                    Tracking bug

                            
https://bugs.chromium.org/p/chromium/issues/detail?id=885175


                                    Estimated milestones

                            108 or 109


                                    Anticipated spec changes

                            We filed multiple issues related to this
                            feature - primarily for all the
                            different types of layout algorithms how
                            to determine their last baseline was
                            underspecified.

                            Scroll container -
                            https://github.com/w3c/csswg-drafts/issues/7660

                            Flexbox -
                            https://github.com/w3c/csswg-drafts/issues/7641

                            Multicol -
                            https://github.com/w3c/csswg-drafts/issues/7639

                            Tables -
                            https://github.com/w3c/csswg-drafts/issues/7655

                            Fieldset -
                            https://github.com/w3c/csswg-drafts/issues/7656

                            (and more).

                            All of these issues have been resolved,
                            and added tests.


                                    Link to entry on the Chrome
                                    Platform Status

                            https://chromestatus.com/feature/5093352798683136


                                    Links to previous Intent discussions

                            Intent to prototype:
                            
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpT5eVmfB6e-adc8exG1ouse25C_0A_2tn3fEVTwh-SetQ%40mail.gmail.com
                            
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/cajl3upt5evmfb6e-adc8exg1ouse25c_0a_2tn3fevtwh-s...@mail.gmail.com>


-- 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 blink-dev+unsubscr...@chromium.org.
                            To view this discussion on the web visit
                            
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpRb9dPstEirAv6pqspLgyab9%2B6bonCEzMrDNumkVH7S0g%40mail.gmail.com
                            
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpRb9dPstEirAv6pqspLgyab9%2B6bonCEzMrDNumkVH7S0g%40mail.gmail.com?utm_medium=email&utm_source=footer>.



-- TAMURA Kent
                        Software Engineer, Google


-- 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
                        blink-dev+unsubscr...@chromium.org.
                        To view this discussion on the web visit
                        
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGH7WqFn76swgKd%3DguZJdXr656Ox5ZOG1SQwDB%2B877L9G08JRw%40mail.gmail.com
                        
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGH7WqFn76swgKd%3DguZJdXr656Ox5ZOG1SQwDB%2B877L9G08JRw%40mail.gmail.com?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
                    blink-dev+unsubscr...@chromium.org.
                    To view this discussion on the web visit
                    
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfVo7bM0GHGuvcwcCNuuAbE0%3DH6tipmcHzsqPybCXFtsKg%40mail.gmail.com
                    
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfVo7bM0GHGuvcwcCNuuAbE0%3DH6tipmcHzsqPybCXFtsKg%40mail.gmail.com?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
                blink-dev+unsubscr...@chromium.org.
                To view this discussion on the web visit
                
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpSeh7ut1KYx-uSMK7_uS2EkVPF8fa-Ku5ucMELx3JfR4Q%40mail.gmail.com
                
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpSeh7ut1KYx-uSMK7_uS2EkVPF8fa-Ku5ucMELx3JfR4Q%40mail.gmail.com?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
                blink-dev+unsubscr...@chromium.org.
                To view this discussion on the web visit
                
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/6a3ebb96-40c9-615c-2c8e-8bf2e844de65%40chromium.org
                
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/6a3ebb96-40c9-615c-2c8e-8bf2e844de65%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 blink-dev+unsubscr...@chromium.org.
            To view this discussion on the web visit
            
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpTgOPZFJcB3DxU7B5Qe0soN%3Do5tV-TcQcntNm8bprc%2BoQ%40mail.gmail.com
            
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJL3UpTgOPZFJcB3DxU7B5Qe0soN%3Do5tV-TcQcntNm8bprc%2BoQ%40mail.gmail.com?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 blink-dev+unsubscr...@chromium.org.
        To view this discussion on the web visit
        
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYctYhJLcFGyPD2uZ1D%3Ds7%2BHjXJe4BBLR43b68hwxiMjRA%40mail.gmail.com
        
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYctYhJLcFGyPD2uZ1D%3Ds7%2BHjXJe4BBLR43b68hwxiMjRA%40mail.gmail.com?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 blink-dev+unsubscr...@chromium.org.
    To view this discussion on the web visit
    
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/a74589bf-80d2-7d80-4680-f47100c24b2d%40chromium.org
    
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/a74589bf-80d2-7d80-4680-f47100c24b2d%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 blink-dev+unsubscr...@chromium.org.
To view this discussion on the web visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/b7ffef98-f487-aabf-529f-82e6c79f0645%40chromium.org.

Reply via email to