SSENSE/vue-carousel

Feature request: Use style classes instead of inline styles

Open

#48 geöffnet am 19. Apr. 2017

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

Beschreibung

There're a few component options regarding styles such as paginationActiveColor, paginationPadding, etc. Since they are rendered as inline styles, it is hard to override them with CSS classes. It's better to set CSS classed for these elements. What do you think?

Contributor Guide

Tech Stack
vuejavascript
Domain
frontend
Issue Type
refactor
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.
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.
mostly clear
Voraussetzungen
basic VueCSS knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Look at the current implementation of the carousel component, likely in a .vue file. Identify all props like `paginationActiveColor`, `paginationPadding` that set inline styles. Propose a mapping to CSS classes and consider how to allow user overrides. Review any existing comments or linked issues for design preferences.