bigcommerce / catalyst

Catalyst - for Composable Commerce
https://catalyst-demo.site
MIT License
117 stars 153 forks source link

fix: slide not in view when element inside is focused #1687

Open hunterbecton opened 6 days ago

hunterbecton commented 6 days ago

What/Why?

Testing

https://github.com/user-attachments/assets/f4b91351-0f8c-428a-a6fa-ffcae2a096ba

linear[bot] commented 6 days ago

VIB-944 Fix focus issue on Slideshow component

vercel[bot] commented 6 days ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
catalyst-latest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 7:49pm
catalyst-soul ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 7:49pm
3 Skipped Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **catalyst** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst/DmMgHfQrT3D727iGeYf8KucqVSAB)) | | | Nov 25, 2024 7:49pm | | **catalyst-au** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-au/HBzJGuSXJndhyugNjZBVNiLFEfa1)) | | | Nov 25, 2024 7:49pm | | **catalyst-uk** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-uk/Gwk8pWEck5wS8DWQHj277V28LRkQ)) | | | Nov 25, 2024 7:49pm |
changeset-bot[bot] commented 6 days ago

⚠️ No Changeset found

Latest commit: 58caf16d1b2bff21ea9eb1326eaf6ed64a887c84

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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