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

[IMPROVEMENT] - into featured-products.tpl it is necessary to set the default slidesPerView for sliderConfig #381

Open
maofree opened this issue Feb 3, 2024 · 2 comments
Labels

Comments

@maofree
Copy link
Sponsor

maofree commented Feb 3, 2024

Description

Hi
I've found a problem with swiper carousel
when the min breakpoint is different from 1, it is necessary to set slidesPerView outside the breakpoint like so

5

if not under 320 the arrows will manage the carousel thinking there is a visible product

bye

Node.js version

v16

php version

8.1

OS and it's version

linux

Browsers

Chrome

Required module/theme

theme

Reproduction steps

1. Go to the homepage
2. reduce the browser's width under 320px
3. push on the arrows
4. now you will see the problem

Logs

No response

@maofree maofree added the bug Something isn't working label Feb 3, 2024
@Oksydan
Copy link
Owner

Oksydan commented Feb 4, 2024

Hi @maofree,

I am not sure should we rly care about browser screen smaller than 320px 🤔
According to screen size stats https://gs.statcounter.com/screen-resolution-stats/mobile/worldwide - 320px isn't even exists in statistics. What about even smaller screen sizes 🤔

@maofree
Copy link
Sponsor Author

maofree commented Feb 4, 2024

Hi
yes, it may be true, it was to inform you about how swiper behaved, but adding a line doesn't change me much anyway

@Oksydan Oksydan removed the bug Something isn't working label Feb 8, 2024
@Oksydan Oksydan changed the title 🐛 [BUG] - [FIX] into featured-products.tpl it is necessary to set the default slidesPerView for sliderConfig [IMPROVEMENT] - into featured-products.tpl it is necessary to set the default slidesPerView for sliderConfig Feb 8, 2024
@Oksydan Oksydan added the 4.0.0 label Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

2 participants