patternfly / patternfly/patternfly-react

Bug - RTL demo cleanup

Ouverte
#12,351 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

PF Team Stale
Langage dominant
TypeScript
Étoiles
862
Forks
392
Merge moyen
4 j 8 h
PR mergées (30 j)
9

Description

Noticed this page could use some updates https://www.patternfly.org/patterns/right-to-left/react-demos/paginated-table/

  • Remove the card wrapping the page content

  • Update probably all of the <Icon /> components to use the inline variant so they inherit the icon color from their parent. You can see the problem when switching to dark theme

  • On mobile the masthead menu goes off screen as seen in Exhibit A

  • The logo shifts away from the hamburger menu in RTL as seen in Exhibit B

References

Exhibit A

image

Exhibit B

image


Jira Issue: PF-3991

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par la démo React de tableau paginé RTL sur la page PatternFly liée et examinez son wrapper de card, les composants Icon, le menu du masthead et la disposition du logo. C’est terminé lorsque la card est supprimée, que les icônes héritent de la couleur de leur parent dans le thème sombre et que le masthead et le logo restent correctement positionnés sur mobile en RTL.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
react, typescript
Domaine
design, frontend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
58/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.