Skip to content

fix(stacks-svelte): shift popovers horizontally#2288

Merged
giamir merged 2 commits into
mainfrom
gbuoncristiani/popover-horizontal-shift
May 25, 2026
Merged

fix(stacks-svelte): shift popovers horizontally#2288
giamir merged 2 commits into
mainfrom
gbuoncristiani/popover-horizontal-shift

Conversation

@giamir
Copy link
Copy Markdown
Contributor

@giamir giamir commented May 25, 2026

Summary

  • Add Floating UI shift middleware to Popover so content can shift horizontally when it would overflow the viewport.
  • Add a regression test for right-edge horizontal overflow.
  • Add a Storybook story for quick visual review.

How to test

  • Run npm run storybook -w packages/stacks-svelte and check Components / Popover / Horizontal Shift.
  • Run cd packages/stacks-svelte && npx web-test-runner src/components/Popover/Popover.test.ts.

@giamir giamir requested a review from a team as a code owner May 25, 2026 11:55
@netlify
Copy link
Copy Markdown

netlify Bot commented May 25, 2026

Deploy Preview for stacks ready!

Name Link
🔨 Latest commit 92a30bd
🔍 Latest deploy log https://app.netlify.com/projects/stacks/deploys/6a1438cd6016220008f078d8
😎 Deploy Preview https://deploy-preview-2288--stacks.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.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 25, 2026

🦋 Changeset detected

Latest commit: 92a30bd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@stackoverflow/stacks-svelte Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@giamir giamir merged commit a84dd91 into main May 25, 2026
17 checks passed
@giamir giamir deleted the gbuoncristiani/popover-horizontal-shift branch May 25, 2026 12:08
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.

1 participant