SSENSE/vue-carousel

Bug: pageChange event gives NaN sometimes with mouse drag

Offen

#408 geöffnet am 26.03.2019

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (513 Forks)batch import
buggood first issuehelp wanted

Repository-Metriken

Stars
 (1.705 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Bug Report

Current Behavior On mouse drag on the last slide, sometimes pageChange event gives NaN while remaining on the same slide where mouse was dragged upon and after that carousel becomes non-operational on mouse drag(slides do not move at all).

Did not face this issue with custom navigation buttons however.

Input Code and steps to reproduce Attaching screenshot of console log, where pageChange event handler logs the page number.

Screen Shot 2019-03-26 at 6 51 42 PM

Expected behavior/code pageChange event shouldn't be triggered, if there is no slide change at all.

Babel Configuration (.babelrc, package.json, cli command)

Contributor Guide

Research-Richtung
Untersuche die Auslösung des pageChange Ereignisses in der Karussell Komponente. Das Problem tritt während des Mausziehens auf der letzten Folie auf. Überprüfe die Zieh Logik und stelle sicher, dass der Seitenindex nur bei einem Folienwechsel ausgelöst wird, nicht bei jeder Ziehbewegung. Überprüfe die Berechnung des Seitenindex, um NaN Werte zu verhindern. Sieh dir die Event Handler der Komponente und den Seitenaktualisierungsmechanismus in Dateien wie Carousel.vue oder ähnlichen an.
Tech Stack
vuejavascript
Domain
frontend
Issue Type
Fehler
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 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
VueJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65