className default is lost when props.className is undefined in UncontrolledTabs
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 45/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript, react
- Domaine
- frontend
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.
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.
- Langage dominant
- JavaScript
- Étoiles
- 3.1k
- Forks
- 455
- Merge moyen
- 5 h 9 min
- PR mergées (30 j)
- 2
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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.
Autres issues de reactjs/react-tabs
-
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
reactjs/react-tabs#584 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
reactjs/react-tabs#571 ·
-
Support hiding of all TabPanelsOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
reactjs/react-tabs#525 · 1 commentaire · 1 réaction ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
reactjs/react-tabs#521 · 3 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
reactjs/react-tabs#507 · 1 commentaire ·
Toutes les issues de reactjs/react-tabs
Issues similaires
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
amponce/archive-movie-browser#354 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
saayam-for-all/webapp#1870 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Imageomics/OpenCite#66 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
chr15m/twiiit.com#20 ·
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
antropos17/Aegis#629 ·
Les mainteneurs répondent en général sous 4 jours