patternfly / patternfly/patternfly-react
Icons - add support for custom icons
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
Is this a new component or an extension of an existing one?
What is the existing component, if any?
Existing, MenuToggle, ToolbarFilter
When using PF tokens to create a custom theme, MenuToggle and ToolbarFilter use default icons in PF that currently cannot be changed via a prop since the prop does not exist, while components like Label, Alert, and Button, offer this functionality.
An alternative option was discussed at today's working session (cc @mcoker), which would include adding support for custom icons directly to react-icons and then importing any custom overrides.
Related: https://github.com/patternfly/patternfly-react/issues/10634
Describe the feature
A clear and concise description of the new feature. What is the expected behavior?
Are there visuals for this feature? If applicable, please include examples for each state and for varying widths
See below. MUI uses a different toggle icon for their menu toggle.
Include screenshots or links to Marvel or other mockups.
MUI:
PF:
Any other information?
Would it be possible to prioritize this work for an upcoming sprint to align with RHOAI priorities for the MUI theming effort @nicolethoen ?
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 les implémentations des composants MenuToggle et ToolbarFilter, puis lisez l’issue #10634 et la discussion sur l’extension de react-icons. Le travail sera considéré comme terminé lorsque les utilisateurs pourront fournir des icônes personnalisées pour les deux composants via une API convenue et documentée, avec un comportement cohérent avec la prise en charge existante des icônes dans Label, Alert et Button.
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
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100