Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rip out all of the beautiful bespoke Popover animation stuff #38392

Merged
merged 1 commit into from
Feb 17, 2023

Commits on Feb 17, 2023

  1. Rip out all of the beautiful bespoke Popover animation stuff

    This isn't part of the landed spec [1], and will be replaced by
    a combination of these five CSSWG issues:
    
    - w3c/csswg-drafts#4441
    - w3c/csswg-drafts#6429
    - w3c/csswg-drafts#8174
    - w3c/csswg-drafts#8189
    - w3c/csswg-drafts#8389
    
    After this CL, you will no longer be able to animate your
    popover like this:
    
    ```
      [popover] {
        opacity: 0;
        transition: opacity 0.2s;
      }
      [popover]:open {
        opacity: 1;
      }
    ```
    
    Instead you'll need to use CSS animations or (eventually) transitions
    and you'll have to explicitly declare the `display` and `top-layer`
    properties:
    
    ```
      transition: opacity 0.2s, display 0.2s, top-layer 0.2s;
    ```
    
    [1] https://html.spec.whatwg.org/multipage/popover.html
    
    Bug: 1307772,1413556
    Change-Id: I4877dd69a06f2624bdb463b065b2e2b66cbf1154
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4225730
    Reviewed-by: David Baron <dbaron@chromium.org>
    Commit-Queue: David Baron <dbaron@chromium.org>
    Commit-Queue: Mason Freed <masonf@chromium.org>
    Auto-Submit: Mason Freed <masonf@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1107048}
    Mason Freed authored and chromium-wpt-export-bot committed Feb 17, 2023
    Configuration menu
    Copy the full SHA
    d99c921 View commit details
    Browse the repository at this point in the history