[whatwg/fullscreen] Rephrase: 3.12 While the first element in fullscreenElements is ... (#144)

https://github.com/whatwg/fullscreen/blob/7ecd70946501abb8692bbf8ce72fa2b8d3ccf3ac/review-drafts/2019-01.bs#L331

IM Humble and not english fluent O, it might be not correct sentence. Easily, there are no condition to exit this While loop. I think writer's purpose was that if first element in "fullscreenElements" is in a nested browsing context: append browsing context container and its ancestors browsing context container recursively.

A implementation: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/fullscreen/fullscreen.cc?q=fullscreen.cc&sq=package:chromium&dr&l=706

I think it should be changed correct sentence to fulfill logical completeness. As I'm not english native, maybe I have misunderstanding on that. Please correct me if I'm wrong.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fullscreen/issues/144

Received on Wednesday, 20 February 2019 06:16:42 UTC