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/6c1e37cc-4acd-4d91-a909-4841a2d57615n%40chromium.org.

Reply via email to