patternfly / patternfly/patternfly-react
Page - change notificationDrawer props to be more generic
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
re: https://github.com/patternfly/patternfly-react/issues/8541#issuecomment-1384624420
There are currently a props on the <Page> component (notificationDrawer and isNotificationDrawerExpanded) that attach a drawer to the page. AFAIK there is no requirement to use a notification drawer there - the props just update the page to support a drawer that appears over the main content area. Assuming we don't want to limit that use of a drawer in the page to notification drawer only, I wonder if it makes sense to rename those props to something more generic so it's clear it can be used for any sort of drawer content.
cc @mcarrano @mceledonia @tlabaj
Jira Issue: PF-141
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 le composant Page et recherchez les props notificationDrawer et isNotificationDrawerExpanded ainsi que leurs utilisations. Confirmez le nommage générique attendu pour le contenu du drawer, mettez à jour les références concernées et vérifiez que le comportement du drawer de Page reste inchangé.
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é
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100