SSENSE/vue-carousel

Feature Request: Allow dynamic number of slides

Open

#268 geöffnet am 20. Aug. 2018

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.705 Stars) (513 Forks)batch import
featuregood first issuehelp wanted

Beschreibung

I have a dynamic number of images I want to display in your carousel. For large screens I want to display up to 8 slides per page. But since the number of images is dynamic I might end up having only 2 images to display. In that case the slide takes up way too much space.

Example: https://codesandbox.io/s/n3mkw7z7q0 I highlighted the background of each slide to easily see how much space it takes up.

It would be great if there was a way to deal with this problem, maybe by being able to set the flex-basis yourself?

Contributor Guide

Tech Stack
vuejavascript
Domain
frontend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Vue.jsCSS flexbox
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
Examine the Carousel component (likely in src/Carousel.vue) to understand current slide width calculation. Implement a prop or reactive calculation that allows the number of slides per page to adjust based on the number of children and container width. Use the provided codesandbox example (https://codesandbox.io/s/n3mkw7z7q0) as a test case. Ensure the solution works with responsive design and existing functionality.