Re: [w3c/editing] Address VK spec review comments. (#317)

@snianu commented on this pull request.



> @@ -189,17 +250,17 @@ <h2>The <dfn>VirtualKeyboard</dfn> Interface</h2>
         <h2>Extensions to the <dfn>ElementContentEditable</dfn> mixin</h2>
         <pre class="idl">
             partial interface ElementContentEditable {
-                [CEReactions, Reflect, ReflectOnly=("auto","manual")] attribute DOMString virtualKeyboardPolicy;
+                [CEReactions] attribute DOMString virtualKeyboardPolicy;

I was getting ReSpec errors so the recommendation (per [this ](https://github.com/w3c/editing/pull/317#issuecomment-885946703)comment) is to just mention it in the definition of the attribute and link it to the IDL. In my case I mentioned it in the `virtualKeyboardPolicy` definition:
> The virtualKeyboardPolicy is an enumerated attribute whose keywords are the string, auto, and manual. The IDL attribute must reflect the respective content attributes of the same name.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/editing/pull/317#discussion_r678879199

Received on Thursday, 29 July 2021 07:00:52 UTC