[webkit-changes] [WebKit/WebKit] 56d7f8: Make fullscreen elements match the :modal pseudo-c...

Tim Nguyen noreply at github.com
Thu Dec 8 10:29:28 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 56d7f894da59a5d699530cb396dbc9400f1836d1
      https://github.com/WebKit/WebKit/commit/56d7f894da59a5d699530cb396dbc9400f1836d1
  Author: Tim Nguyen <ntim at apple.com>
  Date:   2022-12-08 (Thu, 08 Dec 2022)

  Changed paths:
    M LayoutTests/imported/w3c/web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has-expected.txt
    M LayoutTests/imported/w3c/web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has.html
    A LayoutTests/imported/w3c/web-platform-tests/css/selectors/modal-pseudo-class-expected.txt
    A LayoutTests/imported/w3c/web-platform-tests/css/selectors/modal-pseudo-class.html
    M LayoutTests/imported/w3c/web-platform-tests/css/selectors/w3c-import.log
    A LayoutTests/platform/mac-wk1/imported/w3c/web-platform-tests/css/selectors/modal-pseudo-class-expected.txt
    M Source/WebCore/css/SelectorCheckerTestFunctions.h
    M Source/WebCore/dom/Element.cpp

  Log Message:
  -----------
  Make fullscreen elements match the :modal pseudo-class
https://bugs.webkit.org/show_bug.cgi?id=246004
rdar://100782746

Reviewed by Antti Koivisto.

https://w3c.github.io/csswg-drafts/selectors/#modal-state
> For example, the dialog element is :modal when opened with the showModal() API.
> Similarly, a :fullscreen element is also :modal when opened with the requestFullscreen() API, since this prevents interaction with the rest of the page.

We implemented the "prevents interaction with the rest of the page" part in https://commits.webkit.org/253803@main

Relevant CSSWG discussion: https://github.com/w3c/csswg-drafts/issues/7311

Also, we don't need to worry about the modal dialog in fullscreen mode case, since the fullscreen API forbids <dialog> elements.

* LayoutTests/platform/mac-wk1/imported/w3c/web-platform-tests/css/selectors/modal-pseudo-class-expected.txt: Added.
* LayoutTests/imported/w3c/web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has.html:
* LayoutTests/imported/w3c/web-platform-tests/css/selectors/modal-pseudo-class-expected.txt: Added.
* LayoutTests/imported/w3c/web-platform-tests/css/selectors/modal-pseudo-class.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/css/selectors/w3c-import.log:
* Source/WebCore/css/SelectorCheckerTestFunctions.h:
(WebCore::matchesModalPseudoClass):
* Source/WebCore/dom/Element.cpp:
(WebCore::Element::setFullscreenFlag):

Canonical link: https://commits.webkit.org/257572@main




More information about the webkit-changes mailing list