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

🐛 fix(bal-carousel): Scroll Y on mobile blocked and Controls have wrong z index #1254

Closed
1 task done
GinaBiondo-aperto opened this issue Dec 28, 2023 · 2 comments · Fixed by #1265
Closed
1 task done
Assignees
Labels
🐛 bug Something isn't working

Comments

@GinaBiondo-aperto
Copy link
Collaborator

Current Behavior

Somehow the scroll-y prop does not enable vertical scrolling for mobile devices anymore. Scrolling by touch is currently blocked. Also the controls seem to have the wrong z index since scrolled content will overlap the background (not the text content).

Overlapping controls:
Bildschirmfoto 2023-12-28 um 14 25 49

Expected Behavior

The carousel should not block vertical scrolling on the page and the controls should be positioned consistently over the rest of the content.

Steps to Reproduce

Carousel with sticky tab controls and card interface. Currently used for the product packages table.

Version

15.0.2

Browsers

Firefox, Chrome, Safari, Microsoft Edge

Device

Mobile, Tablet

Code Reproduction URL

https://www.baloise.de/de/privatkunden/kfz.html

Additional Information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@GinaBiondo-aperto GinaBiondo-aperto added the 🐛 bug Something isn't working label Dec 28, 2023
@hirsch88 hirsch88 changed the title fix(bal-carousel): Scroll Y on mobile blocked and Controls have wrong z index 🐛 fix(bal-carousel): Scroll Y on mobile blocked and Controls have wrong z index Jan 3, 2024
@hirsch88 hirsch88 self-assigned this Jan 3, 2024
@hirsch88
Copy link
Member

hirsch88 commented Jan 3, 2024

/cib

Copy link
Contributor

github-actions bot commented Jan 3, 2024

Branch fix/issue-1254 created!

github-actions bot added a commit that referenced this issue Jan 3, 2024
hirsch88 added a commit that referenced this issue Jan 3, 2024
…ng z index (#1265)

* Create PR for #1254

* fix(carousel): set z-index for sticky

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Gery Hirschfeld <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
2 participants