Re: [whatwg/fullscreen] Hide popovers when making an element fullscreen (PR #204)

@foolip commented on this pull request.



> @@ -61,8 +61,16 @@ stated otherwise it is unset.
 <p>All <a for=/>documents</a> have an associated <dfn>list of pending fullscreen events</dfn>, which
 is an <a>ordered set</a> of (<a>string</a>, <a>element</a>) <a>tuples</a>. It is initially empty.
 
-<p>To <dfn>fullscreen an <var>element</var></dfn>, set <var>element</var>'s <a>fullscreen flag</a>
-and <a for="top layer">add</a> it to its <a>node document</a>'s <a>top layer</a>.
+<p>To <dfn>fullscreen an <var>element</var></dfn>:
+
+<ol>
+ <li><p>Run <a>hide all popovers</a>.

Also missing the argument:

```suggestion
 <li><p>Run <a>hide all popovers</a> given <var>element</var>'s <a>node document</a>.
```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fullscreen/pull/204#pullrequestreview-1374937257
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fullscreen/pull/204/review/1374937257@github.com>