Awesome! LGTM1. On Thu, Mar 14, 2024 at 1:35 PM 'Stephanie Zhang' via blink-dev < blink-dev@chromium.org> wrote:
> Thanks for clarifying! Updated the Chrome Status' "Finch Feature Name" > field to kWritingSuggestions and removed the "Non-finch justification" > field. > > On Wednesday, March 13, 2024 at 9:20:57 PM UTC-7 Domenic Denicola wrote: > >> On Thu, Mar 14, 2024 at 12:54 PM 'Stephanie Zhang' via blink-dev < >> blin...@chromium.org> wrote: >> >>> We do have a runtime feature flag 'WritingSuggestions >>> <https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5?q=writingsuggestions>'. >>> We didn't think a Finch Trial was necessary, as the bulk of the changes were >>> just adding the attribute and IDL functions >>> <https://chromium-review.googlesource.com/c/chromium/src/+/5247315>. >>> Since everything is implemented on the blink side, is a Finch feature flag >>> still necessary? If it is, then I'll add that flag :) >> >> >> A runtime feature flag automatically generates >> <https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5;l=121-125;drc=36168a902bb7a33bfc8b46ad1f4ef6672872ad6d> >> a Finch flag, unless you turn that off :). So I think this is just a matter >> of updating the Chrome Status entry. >> >> >>> >>> On Wednesday, March 13, 2024 at 6:55:48 PM UTC-7 Domenic Denicola wrote: >>> >>>> >>>> >>>> On Thursday, March 14, 2024 at 2:08:28 AM UTC+9 Stephanie Zhang wrote: >>>> >>>> *Contact emails* >>>> *sa...@microsoft.com*, *dan...@microsoft.com*, >>>> *stephanie.zh...@microsoft.com* >>>> >>>> *Explainer* >>>> >>>> *https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/WritingSuggestions/explainer.md* >>>> <https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/WritingSuggestions/explainer.md> >>>> >>>> *Specification* >>>> >>>> *https://html.spec.whatwg.org/multipage/interaction.html#writing-suggestions* >>>> <https://html.spec.whatwg.org/multipage/interaction.html> >>>> >>>> *Summary* >>>> UAs are starting to provide writing suggestions to users as they type >>>> on various editable fields across the web. While this is generally useful >>>> for users, there are cases when developers may want to turn off UA-provided >>>> writing assistance, such as extensions or sites that wish to provide >>>> similar functionality on their own. To that end, developers need a solution >>>> that would turn on/off UA-provided writing assistance. The new attribute >>>> 'writingsuggestions' has values 'true'/'false' that would allow developers >>>> to turn on/off browser-provided writing suggestions. The attribute's state >>>> for an element can also be inherited from ancestor elements, thereby >>>> allowing developers to control this functionality at a per-element or >>>> per-document/sub-document scale. >>>> >>>> >>>> *Blink component* >>>> *Blink>Editing* >>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EEditing> >>>> >>>> *TAG review* >>>> *https://github.com/w3ctag/design-reviews/issues/924* >>>> <https://github.com/w3ctag/design-reviews/issues/924> >>>> >>>> *TAG review status* >>>> Issues addressed >>>> >>>> *Risks* >>>> >>>> >>>> *Interoperability and Compatibility* >>>> None >>>> >>>> >>>> *Gecko*: No signal ( >>>> *https://github.com/mozilla/standards-positions/issues/855* >>>> <https://github.com/mozilla/standards-positions/issues/855>) >>>> >>>> *WebKit*: In development ( >>>> *https://github.com/WebKit/standards-positions/issues/308* >>>> <https://github.com/WebKit/standards-positions/issues/308>) WebKit >>>> Implementation PR: *https://github.com/WebKit/WebKit/pull/24051* >>>> <https://github.com/WebKit/WebKit/pull/24051> >>>> >>>> *Web developers*: No signals >>>> >>>> *Other signals*: >>>> >>>> *Ergonomics* >>>> None >>>> >>>> >>>> *Activation* >>>> None >>>> >>>> >>>> *Security* >>>> None >>>> >>>> >>>> *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* >>>> None >>>> >>>> >>>> *Will this feature be supported on all six Blink platforms (Windows, >>>> Mac, Linux, ChromeOS, Android, and Android WebView)?* >>>> Yes >>>> Attribute is available on all platforms. >>>> >>>> >>>> *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/html/editing/editing-0/writing-suggestions/writingsuggestions.html?label=master&label=experimental&aligned* >>>> <https://wpt.fyi/results/html/editing/editing-0/writing-suggestions/writingsuggestions.html?label=master&label=experimental&aligned> >>>> >>>> >>>> *Flag name on chrome://flags* >>>> None >>>> >>>> *Finch feature name* >>>> None >>>> >>>> >>>> Per the flag guidelines >>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/flag_guarding_guidelines.md>, >>>> all new features are required to be placed behind a Finch feature flag >>>> (i.e. base::Feature flag). Can you ensure this is done and update the >>>> Chrome Status entry? >>>> >>>> >>>> >>>> *Non-finch justification* >>>> No finch trial needed. >>>> >>>> >>>> *Requires code in //chrome?* >>>> False >>>> >>>> *Estimated milestones* >>>> Shipping on desktop 124 >>>> >>>> Shipping on Android 124 >>>> >>>> Shipping on WebView 124 >>>> >>>> >>>> >>>> *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).* >>>> None >>>> >>>> *Link to entry on the Chrome Platform Status* >>>> *https://chromestatus.com/feature/5153375153029120* >>>> <https://chromestatus.com/feature/5153375153029120> >>>> >>>> *Links to previous Intent discussions* >>>> Intent to prototype: >>>> *https://groups.google.com/a/chromium.org/g/blink-dev/c/rHyRCx-hJhE* >>>> <https://groups.google.com/a/chromium.org/g/blink-dev/c/rHyRCx-hJhE> >>>> 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+...@chromium.org. >>> To view this discussion on the web visit >>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/8d6a0046-1a9b-44a4-9403-51067ca119fen%40chromium.org >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/8d6a0046-1a9b-44a4-9403-51067ca119fen%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/adc944d3-853d-4964-9408-52b4014cc561n%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/adc944d3-853d-4964-9408-52b4014cc561n%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/CAM0wra_CmAuu_4GVUtG%2BdNxLb8mB4oAXNXZ6PkGmP2jW-F0bxA%40mail.gmail.com.