className default is lost when props.className is undefined in UncontrolledTabs
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 3.1k
- Forks
- 455
- Merge moyen
- 5 h 9 min
- PR mergées (30 j)
- 2
Description
In the UncontrolledTabs component, default props are merged using object spreading:
const { className, ...attributes } = { ...defaultProps, ...props, };
Even the defaultProps.className is defined ('react-tabs') and props.className is explicitly undefined, the spread operation will overwrite the default and set className to undefined.
As a result, the default class name is lost. It's unclear whether this behavior is intended, but it can lead to unexpected issues, especially when upgrading from older versions where the default class was always applied. Projects relying on the default 'react-tabs' class for styling or selectors may break silently.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le composant UncontrolledTabs et examinez l’expansion d’objet de default-props, en reproduisant le cas où props.className est explicitement undefined. Confirmez le comportement attendu pour la classe "react-tabs" par défaut et vérifiez que la classe reste appliquée sans perturber un className fourni explicitement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, react
- Domaine
- frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100