Skip to content

Add padding to demo elements, mark non-working example#423

Merged
jamesnw merged 1 commit into
mainfrom
demo-cleanup
Jun 27, 2026
Merged

Add padding to demo elements, mark non-working example#423
jamesnw merged 1 commit into
mainfrom
demo-cleanup

Conversation

@jamesnw

@jamesnw jamesnw commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Description

Some examples were appearing different due to lack of space around a containing block, and the changes from safe alignment here- https://anchor-positioning.oddbird.net/position-area#no-block-space

I also noted an example that no longer works in polyfilled and non-polyfilled environments.

@jamesnw jamesnw requested a review from jgerigmeyer June 26, 2026 14:05
@netlify

netlify Bot commented Jun 26, 2026

Copy link
Copy Markdown

Deploy Preview for anchor-position-wpt canceled.

Name Link
🔨 Latest commit 0327bca
🔍 Latest deploy log https://app.netlify.com/projects/anchor-position-wpt/deploys/6a3e870c1bd9700008f54b97

@netlify

netlify Bot commented Jun 26, 2026

Copy link
Copy Markdown

Deploy Preview for anchor-polyfill ready!

Name Link
🔨 Latest commit 0327bca
🔍 Latest deploy log https://app.netlify.com/projects/anchor-polyfill/deploys/6a3e870c11a79f00080584bd
😎 Deploy Preview https://deploy-preview-423--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 project configuration.

@jgerigmeyer jgerigmeyer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also noted an example that no longer works in polyfilled and non-polyfilled environments.

Worth filing an issue for that separately?

@jamesnw

jamesnw commented Jun 27, 2026

Copy link
Copy Markdown
Contributor Author

I also noted an example that no longer works in polyfilled and non-polyfilled environments.

Worth filing an issue for that separately?

It's this issue - #279, although the demo seems even more broken than initially reported.

@jamesnw jamesnw merged commit 76e34c9 into main Jun 27, 2026
13 checks passed
@jamesnw jamesnw deleted the demo-cleanup branch June 27, 2026 01:12
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.

2 participants