- From: Tab Atkins Jr. <notifications@github.com>
- Date: Tue, 16 May 2023 16:34:57 -0700
- To: whatwg/fullscreen <fullscreen@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 16 May 2023 23:35:03 UTC
@tabatkins commented on this pull request. > <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>, set <var>element</var>'s <a>fullscreen flag</a>. +If <var>element</var> is not currently [=in the top layer=] of its <a>node document</a>, Okay, I've gone ahead and applied this - when fullscreening, we first request removal *then* add it, so it'll cycle to the end if it was already there. -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/fullscreen/pull/223#discussion_r1195781745 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/fullscreen/pull/223/review/1429579063@github.com>
Received on Tuesday, 16 May 2023 23:35:03 UTC