patternfly / patternfly/react-data-view
DataViewToolbar: Pagination misaligned from rest of toolbar
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 4
- Forks
- 21
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Hey PatternFly team!😁
I was recently working with the DataViewToolbar component attempting to align the toolbar with the pagination in the same row. Currently, when passing a pagination prop to the component the pagination is not aligned in the same row as the rest of the toolbar. Example (and this same behavior is displayed in the examples in the component's docs):
PatternFly's toolbar design guidelines recommend these be aligned in the same row.
After some investigation, I found the pagination is wrapped in a div that uses flex with a set width of 100%. This causes the pagination to render in its own row distinct from the rest of the toolbar. Link to code referenced
Please implement a fix that aligns the pagination in the same row as the rest of the toolbar. Thank you!😄
Jira Issue: PF-4529
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 dans packages/module/src/DataViewToolbar/DataViewToolbar.tsx, au niveau du wrapper de pagination mentionné dans l’issue, puis comparez le comportement du composant avec les exemples de la documentation de DataViewToolbar. Confirmez la correction en vérifiant que la pagination est rendue sur la même ligne que le reste du contenu de la toolbar, conformément aux directives de conception de la toolbar.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 78/100