SSENSE/vue-carousel

Feature Request: Add prop to position navigation dots above or below slides

Open

#313 geöffnet am 7. Nov. 2018

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (1.705 Stars) (513 Forks)batch import
featurehelp wanted

Beschreibung

Is your feature request related to a problem? Sometimes pagination dots are below the fold for tall carousels and/or small displays

Describe the solution you'd like Would we add an option to allow pagination dots at the top and/or the bottom? My current use case would require both top and bottom.

Contributor Guide

Tech Stack
vuejavascriptcsshtml
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.
mostly clear
Voraussetzungen
basic understanding of Vue propsfamiliarity with carousel component structure
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
The issue requests a prop to control positioning of pagination dots (top or bottom). Review the current pagination component at `Pagination.vue` and the main carousel logic in `Carousel.vue`. Check the 4 comments for any additional discussion on implementation details. The maintainers have not responded, so a proposed implementation should include a new prop, say `paginationPosition`, and adjust the CSS accordingly. Ensure backward compatibility, e.g., default to bottom.