LGTM3 On Wed, Sep 6, 2023 at 5:49 PM Alex Russell <slightly...@chromium.org> wrote:
> Was corrected in the API OWNERS meeting re: TAG review; apologies for the > noise. LGTM2 > > On Wednesday, September 6, 2023 at 8:45:28 AM UTC-7 Alex Russell wrote: > >> Why is there no TAG review filed here? >> >> On Wednesday, September 6, 2023 at 8:42:23 AM UTC-7 Philip Jägenstedt >> wrote: >> >>> Thanks for adding that test Joey! >>> >>> >>> https://staging.wpt.fyi/results/css/selectors/valid-invalid-form-fieldset.html?label=pr_head&max-count=1&pr=41801 >>> shows that it also passes on Firefox. Not sure why Safari didn't run, but >>> that's not your fault and not something to block on. >>> >>> LGTM1, and thanks for working on this! >>> >>> On Mon, Sep 4, 2023 at 3:46 PM Joey Arhar <jar...@chromium.org> wrote: >>> >>>> > > I see that https://drafts.csswg.org/selectors-4/#issue-df919919 >>>> states that this and the :invalid/:valid flavors should apply to forms and >>>> fieldset elements. It doesn't look like the WPTs test for that - what do we >>>> do for those elements, and do you know if it's interoperable? >>>> > >>>> > We don't have WPTs at the moment. Based on my testing, it looks like >>>> firefox chrome and safari all don't apply :user-valid or :user-invalid to >>>> form elements. I opened a spec issue to discuss: >>>> https://github.com/w3c/csswg-drafts/issues/9257 >>>> >>>> I am adding WPTs here: >>>> https://chromium-review.googlesource.com/c/chromium/src/+/4839394 >>>> I think that we have rough consensus in the spec issue as well to not >>>> apply :user-valid or :user-invalid to form or fieldset elements. >>>> >>>> On Wed, Aug 30, 2023 at 5:15 PM Daniel Bratell <bratel...@gmail.com> >>>> wrote: >>>> >>>>> I think those are the ones that are hard to manually trigger while >>>>> working in the debugger. >>>>> >>>>> /Daniel >>>>> On 2023-08-26 19:30, PhistucK wrote: >>>>> >>>>> I guess all of them would be good. Not really why only a few >>>>> pseudo-classes are listed there... >>>>> >>>>> ☆*PhistucK* >>>>> >>>>> >>>>> On Sat, Aug 26, 2023 at 6:18 PM Joey Arhar <jar...@chromium.org> >>>>> wrote: >>>>> >>>>>> Sure I can try setting up the force element state feature for it. >>>>>> >>>>>> > along with other form-related ones >>>>>> >>>>>> Any ones you have in mind? I could try to do them all at once >>>>>> >>>>>> On Sat, Aug 26, 2023 at 10:00 AM PhistucK <phist...@gmail.com> wrote: >>>>>> >>>>>>> Sounds good! >>>>>>> >>>>>>> > Debuggability >>>>>>> >>>>>>> > These new pseudo-classes will be supported by the DevTools styles >>>>>>> sidebar automatically, just like every other pseudo-class. >>>>>>> >>>>>>> >>>>>>> Can it (along with other form-related ones, I guess) be added to the >>>>>>> list of toggle-able pseudo classes (shown when you click on the ":hov" >>>>>>> button)? >>>>>>> [image: image.png] >>>>>>> >>>>>>> >>>>>>> >>>>>>> ☆*PhistucK* >>>>>>> >>>>>>> >>>>>>> On Sat, Aug 26, 2023 at 9:14 AM Joey Arhar <jar...@chromium.org> >>>>>>> wrote: >>>>>>> >>>>>>>> Contact emails jar...@chromium.org >>>>>>>> >>>>>>>> Explainer None >>>>>>>> >>>>>>>> Specification https://drafts.csswg.org/selectors-4/#user-pseudos >>>>>>>> >>>>>>>> Summary >>>>>>>> >>>>>>>> The :user-invalid and the :user-valid pseudo-classes represent an >>>>>>>> element with incorrect or correct input, respectively, but only after >>>>>>>> the >>>>>>>> user has significantly interacted with it. This is similar to :valid >>>>>>>> and >>>>>>>> :invalid, but with the added constraint that these pseudo-classes only >>>>>>>> match after the user has interacted with the element. >>>>>>>> >>>>>>>> >>>>>>>> Blink component Blink>CSS >>>>>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ECSS> >>>>>>>> >>>>>>>> TAG review None >>>>>>>> >>>>>>>> TAG review status Not applicable >>>>>>>> >>>>>>>> Risks >>>>>>>> >>>>>>>> >>>>>>>> Interoperability and Compatibility >>>>>>>> >>>>>>>> There is no interop/compat risks because this is a new feature that >>>>>>>> has already been implemented by safari and firefox and has WPTs. >>>>>>>> >>>>>>>> >>>>>>>> *Gecko*: Shipped/Shipping >>>>>>>> >>>>>>>> *WebKit*: Shipped/Shipping >>>>>>>> >>>>>>>> *Web developers*: No signals >>>>>>>> >>>>>>>> *Other signals*: >>>>>>>> >>>>>>>> Ergonomics >>>>>>>> >>>>>>>> This feature will not be used in tandem with other platform APIs. >>>>>>>> The default usage of this API will not make it hard for chrome to >>>>>>>> maintain >>>>>>>> good performance. >>>>>>>> >>>>>>>> >>>>>>>> Activation >>>>>>>> >>>>>>>> It will not be challenging for developers to use this feature >>>>>>>> immediately. There is already an MDN article for this feature, so I >>>>>>>> don't >>>>>>>> think that we need additional outreach. >>>>>>>> >>>>>>>> >>>>>>>> Security >>>>>>>> >>>>>>>> There are no security risks for this feature. >>>>>>>> >>>>>>>> >>>>>>>> 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? >>>>>>>> >>>>>>>> None >>>>>>>> >>>>>>>> >>>>>>>> Debuggability >>>>>>>> >>>>>>>> These new pseudo-classes will be supported by the DevTools styles >>>>>>>> sidebar automatically, just like every other pseudo-class. >>>>>>>> >>>>>>>> >>>>>>>> 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 >>>>>>>> >>>>>>>> Flag name on chrome://flags UserValidUserInvalid >>>>>>>> >>>>>>>> Finch feature name UserValidUserInvalid >>>>>>>> >>>>>>>> Requires code in //chrome? False >>>>>>>> >>>>>>>> Availability expectation This feature is already being shipped by >>>>>>>> safari and firefox, so it will be available on the web platform >>>>>>>> mainline as >>>>>>>> soon as we launch it. >>>>>>>> >>>>>>>> Adoption expectation This feature will be considered the best >>>>>>>> practice for its use case as soon as we launch it. >>>>>>>> >>>>>>>> Adoption plan This is already implemented in safari and firefox, >>>>>>>> so we don't need to do anything in order to gain adoption of this >>>>>>>> feature. >>>>>>>> >>>>>>>> Sample links >>>>>>>> https://developer.mozilla.org/en-US/docs/Web/CSS/:user-valid >>>>>>>> >>>>>>>> Estimated milestones >>>>>>>> Shipping on desktop 118 >>>>>>>> DevTrial on desktop 118 >>>>>>>> Shipping on Android 118 >>>>>>>> DevTrial on Android 118 >>>>>>>> Shipping on WebView 118 >>>>>>>> >>>>>>>> Anticipated spec changes >>>>>>>> >>>>>>>> Open questions about a feature may be a source of future web compat >>>>>>>> or interop issues. Please list open issues (e.g. links to known github >>>>>>>> issues in the project for the feature specification) whose resolution >>>>>>>> may >>>>>>>> introduce web compat/interop risk (e.g., changing to naming or >>>>>>>> structure of >>>>>>>> the API in a non-backward-compatible way). >>>>>>>> There are no anticipated spec changes. >>>>>>>> >>>>>>>> Link to entry on the Chrome Platform Status >>>>>>>> https://chromestatus.com/feature/5132477781245952 >>>>>>>> >>>>>>>> This intent message was generated by Chrome Platform Status >>>>>>>> <https://chromestatus.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/CAK6btwKVw_WLSEZ348JyUXHVXfrNOBD7DN1U5svUkQ%3D1TLADFg%40mail.gmail.com >>>>>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAK6btwKVw_WLSEZ348JyUXHVXfrNOBD7DN1U5svUkQ%3D1TLADFg%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/CABc02_Kdx1qNJkwPADG4UaN%3DAVAPkHR9FWu%2BBExActRbYgSkqg%40mail.gmail.com >>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CABc02_Kdx1qNJkwPADG4UaN%3DAVAPkHR9FWu%2BBExActRbYgSkqg%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/CAK6btwJJA4Doh_On-cqQ-hk5%2BFFYvOOTTqZh5T8goeVivN189g%40mail.gmail.com >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAK6btwJJA4Doh_On-cqQ-hk5%2BFFYvOOTTqZh5T8goeVivN189g%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/6fa8b0fa-d2c5-4d1b-9d5a-14dc9a49fa31n%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/6fa8b0fa-d2c5-4d1b-9d5a-14dc9a49fa31n%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/CAL5BFfXoNON5hvmbz9u__xM%3DBBYUTr-HznW5dhYQyejMHzsO2Q%40mail.gmail.com.