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

WIP: Allow unapplying and reapplying polyfill. #123

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

jgerigmeyer
Copy link
Member

@jgerigmeyer jgerigmeyer commented Sep 22, 2023

This adds a top-level restore function, which attempts to un-apply the polyfill's effects. This allows then re-applying the polyfill, if desired.

This may not be the best solution for #91, but it seemed like a reasonable first step, and might have value on its own as well.

@netlify
Copy link

netlify bot commented Sep 22, 2023

Deploy Preview for anchor-polyfill ready!

Name Link
🔨 Latest commit 75a0189
🔍 Latest deploy log https://app.netlify.com/sites/anchor-polyfill/deploys/66996d520a39b5000807c884
😎 Deploy Preview https://deploy-preview-123--anchor-polyfill.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Sep 22, 2023

Deploy Preview for anchor-position-wpt canceled.

Name Link
🔨 Latest commit a34baff
🔍 Latest deploy log https://app.netlify.com/sites/anchor-position-wpt/deploys/650db6752eb67d000809603e

@github-actions
Copy link
Contributor

@jgerigmeyer
Copy link
Member Author

@jgerigmeyer jgerigmeyer linked an issue Jul 18, 2024 that may be closed by this pull request
* main: (120 commits)
  Add high level architecture section to docs
  loosen workflow run limits
  Comment out broken WPT report
  v0.1.0
  upgrade deps
  Skip and comment failing test
  chore(deps-dev): Bump the dev group with 4 updates
  chore(deps): Bump @floating-ui/dom from 1.6.5 to 1.6.7 in the prod group
  Update test to match WPT (failing: https://github.com/oddbird/css-anchor-positioning/issues/103\#issuecomment-2192225109)
  Update validity test assumptions
  Remove unneeded Polyfill
  Update README.md
  review
  Address review
  use generateCSS fn
  remove duplicate fn
  consistent type imports
  update comment
  chore(deps-dev): Bump the dev group across 1 directory with 11 updates
  chore(deps): Bump @types/css-tree from 2.3.7 to 2.3.8 in the prod group
  ...
Copy link

netlify bot commented Jul 18, 2024

Deploy Preview for anchor-position-wpt canceled.

Name Link
🔨 Latest commit 75a0189
🔍 Latest deploy log https://app.netlify.com/sites/anchor-position-wpt/deploys/66996d5232b03b0008bfb2a0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

applying the polyfill to dynamic elements
1 participant