shoelace-style/webawesome

Dynamically adding/removing carousel slides with the `loop` property is glitchy

Open

#2 173 ouverte le 16 mars 2026

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)TypeScript (157 forks)github user discovery
bughelp wanted

Métriques du dépôt

Stars
 (1 104 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Describe the bug

When adding slides to a looping sl-carousel dynamically, the clone slides present challenges.

To Reproduce

Steps to reproduce the behavior:

  1. Go to the sl-carousel docs page
  2. Scroll to the add/remove example
  3. Modify the dom in your browser to add the loop attribute to the sl-carousel
  4. Try to use the example to add/remove slides.

Additional information

This could probably be solved by having the clone slides live outside the slot.

Pseudocode alert!

render() {
...
  if (loop) {
    <sl-carousel-clone></sl-carousel-clone>
  }
  <slot></slot>
  if (loop) {
    <sl-carousel-clone></sl-carousel-clone>
  }
...
}

Guide contributeur

Direction de recherche
Inspectez comment les diapositives clonées sont actuellement rendues dans le slot du carrousel et envisagez de les déplacer hors du slot pour éviter les interférences avec l'ajout/la suppression dynamique de diapositives.
Stack technique
typescript
Domaine
frontendweb dev
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75