patternfly / patternfly/patternfly-react
Alert - remove hasAnimations prop during breaking change
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 862
- Forks
- 392
- Merge moyen
- 4 j 8 h
- PR mergées (30 j)
- 9
Description
Describe the enhancement or change
Followup to animations PR https://github.com/patternfly/patternfly-react/pull/11495 where hasAnimations was added as beta, with the intent being it'd be used for testing purposes only. In the next breaking change we would expect consumers to update tests to reflect animations within an AlertGroup and should remove the prop.
Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?
Any other information?
Jira Issue: PF-2207
Guide de contribution
Ouvrir le guide de contribution
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 examiner hasAnimations dans le composant AlertGroup et ses tests, en utilisant la PR d’animations #11495 comme contexte. Supprimez la prop beta lors du Breaking Change et mettez à jour les tests associés afin que le comportement d’animation attendu de AlertGroup corresponde à la nouvelle API.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react, typescript
- Domaine
- frontend
- Type d'issue
- Refactorisation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100