LGTM3

Would like to see a concrete plan for expanding this method to other 
media-loading elements (<link>, <style>, etc.)

On Thursday, September 23, 2021 at 10:50:17 AM UTC-7 Daniel Bratell wrote:

> LGTM2
>
> /Daniel
> On 2021-09-23 12:06, Yoav Weiss wrote:
>
> LGTM1
>
> On Wed, Sep 22, 2021 at 5:21 AM Tsuyoshi Horo <h...@chromium.org> wrote:
>
>> Contact emails 
>>
>> h...@chromium.org
>>
>>  
>> Explainer 
>>
>> https://github.com/horo-t/explainers/blob/main/script_element_supports.md
>>
>>  
>> Specification 
>>
>> https://html.spec.whatwg.org/multipage/scripting.html#dom-script-supports
>>
>  
>> Design docs 
>>
>> https://github.com/horo-t/explainers/blob/main/script_element_supports.md
>>
>>  
>> Summary 
>>
>> Provides a unified way to detect new features that use script elements.
>>
>>  
>>
>>  
>> Blink component 
>>
>> Blink>HTML>Script 
>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EHTML%3EScript>
>>
>>  
>> TAG review 
>>
>> https://github.com/w3ctag/design-reviews/issues/674
>>
>>  
>> TAG review status 
>>
>> Issues addressed
>>
>
> It should be noted that there was disagreement 
> <https://github.com/w3ctag/design-reviews/issues/674#issuecomment-920189749> 
> between the TAG and the spec authors on the naming. 
> While I lean towards the call the spec authors made, the need for better 
> JS feature detection is something I repeatedly hear both from the TAG and 
> <https://docs.google.com/document/d/1zxs8kZVUtyqWzRr9SaXGOwrRAOBOYTMRs4MZJ9ahK98/edit?resourcekey=0-xW4ilB9FrmJFkRIOumTTPQ#heading=h.c0uts5ftkk58>
>  
> others <https://github.com/whatwg/html/issues/4432>. Orthogonal to this 
> intent, this seems like something we should eventually tackle.
>
>  
>> Risks Interoperability and Compatibility 
>>
>> This method provides a synchronous way of feature detections. But for 
>> unsupported browsers, developers need to use an asynchronous way as 
>> discussed at https://github.com/WICG/import-maps/issues/171.
>>
>
> Is it worthwhile to document the desired code patterns we want developers 
> to use? (e.g. test is `supports()` exists and fallback to the async method 
> otherwise)
>  
>
>>  
>>
>>  
>>
>> Gecko: Worth prototyping (
>> https://github.com/mozilla/standards-positions/issues/576)
>>
>>  
>>
>> WebKit: No signal (
>> https://lists.webkit.org/pipermail/webkit-dev/2021-September/031979.html)
>>
>>  
>>
>> Web developers: Positive The developer feedback on the spec issue (
>> https://github.com/whatwg/html/issues/6472) was positive.
>>
>>  
>> Ergonomics 
>>
>> No known ergonomics risks.
>>
>>  
>>
>>  
>> Activation 
>>
>> No known activation risks. This method is easy to use.
>>
>>  
>>
>>  
>> Security 
>>
>> No known security risks.
>>
>>  
>>
>>  
>> Debuggability 
>>
>> Developers can call this method from DevTools's console.
>>
>>  
>>
>>  
>> Is this feature fully tested by web-platform-tests 
>> <https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md>
>> ? 
>>
>> Yes
>>
>>  
>> DevTrial instructions 
>>
>>
>> https://github.com/horo-t/explainers/blob/main/script_element_supports_how_to.md
>>
>>  
>> Flag name 
>>
>> ScriptElementSupports
>>
>>  
>> Requires code in //chrome? 
>>
>> False
>>
>>  
>> Tracking bug 
>>
>> https://bugs.chromium.org/p/chromium/issues/detail?id=1245528
>>
>>  
>> Sample links 
>>
>> https://horo-t.github.io/explainers/script_element_supports_sample.html
>>
>>  
>> Estimated milestones  DevTrial on desktop 95 DevTrial on android 95 DevTrial 
>> on Webview 95   
>>
>>  
>> Link to entry on the Chrome Platform Status 
>>
>> https://www.chromestatus.com/feature/5712146835963904
>>
>>  
>> Links to previous Intent discussions 
>>
>> Intent to prototype: 
>> https://groups.google.com/a/chromium.org/g/blink-dev/c/evk2qgsekYk/m/WtdE_XplBQAJ
>>
>> Ready for Trial: 
>> https://groups.google.com/a/chromium.org/g/blink-dev/c/-sE2GpCrG6Y/m/UsHXyVW-CAAJ
>>
>>  
>>
>>  
>> This intent message was generated by Chrome Platform Status 
>> <https://www.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/CADk0S-Vi%2BhCDOA2y0%2BfUU-dt60_ySvSY0-MjDyGTMi8oHcrm9A%40mail.gmail.com
>>  
>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CADk0S-Vi%2BhCDOA2y0%2BfUU-dt60_ySvSY0-MjDyGTMi8oHcrm9A%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/CAL5BFfV1cuHkDVpSbirXFPc7Ej5cX4hcpQumcwkK39988%3DDRyg%40mail.gmail.com
>  
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfV1cuHkDVpSbirXFPc7Ej5cX4hcpQumcwkK39988%3DDRyg%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/545e1e7f-7285-41fa-b16a-6918021593b8n%40chromium.org.

Reply via email to