On Fri, Sep 24, 2021 at 5:42 AM Tsuyoshi Horo <h...@google.com> wrote:
> Thank you very much for LGTMs! > > > >> RisksInteroperability 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) >> > > Yes. Sure. I updated this doc > <https://horo-t.github.io/explainers/script_element_supports_how_to.html> > to explain how to detect the importmap support > without HTMLScriptElement.supports(type) method. > That's great! It would be similarly great if that finds itself into developer-facing documentation (MDN, web.dev, etc). Searching around, I can't find any for neither this nor import maps :/ > > > > On Fri, Sep 24, 2021 at 4:24 AM Domenic Denicola <dome...@chromium.org> > wrote: > >> >> >> On Thu, Sep 23, 2021 at 3:15 PM Alex Russell <slightly...@chromium.org> >> wrote: >> >>> LGTM3 >>> >>> Would like to see a concrete plan for expanding this method to other >>> media-loading elements (<link>, <style>, etc.) >>> >> >> <link> already supports this via linkEl.relList.supports("foo"). The >> different API shape is necessary because rel="" can contain multiple >> space-separated tokens, and it also generalizes with other DOMTokenLists >> that benefit from supports() like iframeEl.sandbox.supports("foo"). >> >> <style> currently only supports CSS, but I agree that if we were to ever >> add multiple stylesheet languages to the platform then we'd want an API >> like this on HTMLStyleElement. >> >> >>> >>> 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 >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/545e1e7f-7285-41fa-b16a-6918021593b8n%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/CAL5BFfVrROv6JHCb3r48QmZj4v4MOxbA%2B2Zqix%2BmJs_s01SL_Q%40mail.gmail.com.