Skip to content

Handle positioned elements with fixed positioning#413

Open
jamesnw wants to merge 4 commits into
mainfrom
strategy
Open

Handle positioned elements with fixed positioning#413
jamesnw wants to merge 4 commits into
mainfrom
strategy

Conversation

@jamesnw

@jamesnw jamesnw commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Description

We were always using the absolute strategy for Floating UI, which meant that fixed position elements were not positioned correctly. This makes the strategy dynamic, based on the positioning of the positioned element.

Related Issue(s)

This relates to #350, and we should check if it fixes that.

Steps to test/reproduce

Check out position area and anchor examples

@netlify

netlify Bot commented Jun 15, 2026

Copy link
Copy Markdown

Deploy Preview for anchor-position-wpt canceled.

Name Link
🔨 Latest commit 76ba65d
🔍 Latest deploy log https://app.netlify.com/projects/anchor-position-wpt/deploys/6a30091833976c0008c37c94

@jamesnw jamesnw requested a review from jgerigmeyer June 15, 2026 14:09
@netlify

netlify Bot commented Jun 15, 2026

Copy link
Copy Markdown

Deploy Preview for anchor-polyfill ready!

Name Link
🔨 Latest commit 76ba65d
🔍 Latest deploy log https://app.netlify.com/projects/anchor-polyfill/deploys/6a3009183fded60008c50858
😎 Deploy Preview https://deploy-preview-413--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.

Works for me! Interesting, in current Chrome (no polyfill) the popover does not scroll for me, but in current FF (also no polyfill) it does scroll with the page. Unrelated, but notable.

Comment thread index.html
}</code></pre>
</section>

<section id="fixed-target" class="demo-item">

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.

Suggested change
<section id="fixed-target" class="demo-item">
<section id="fixed" class="demo-item">

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