SSENSE/vue-carousel

Bug: When items are loaded asynchronously the carousel starts at the last pagination dot

Open

#188 geöffnet am 24. Apr. 2018

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

Beschreibung

If you add in the slides with a v-for and get the data inside those async, like for example an AJAX request then the pagination starts at the last dot.

Here's a JSFiddle to illustrate it https://jsfiddle.net/mor7m8s8/1/

I've used a set timeout to simulate the delay but it occurs just the same with an AJAX request.

Contributor Guide

Tech Stack
vuejavascript
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
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
Basic Vue knowledgeUnderstanding of async data loading
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
The bug occurs because the pagination component initializes its active index before async data (slides) are loaded. Investigate the carousel's pagination component logic in the repository, specifically how it sets the initial active dot when slides are added dynamically via v for with async data. Look at the component's mounted hook or watchers on the slides prop. The provided JSFiddle (https://jsfiddle.net/mor7m8s8/1/) simulates the issue with a timeout; reproduce and trace the index assignment. The fix likely involves resetting the pagination index or reacting to slides changes properly.