patternfly / patternfly/patternfly-react
[Spike] - investigate friendly names for all icon sets
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
Instead of needing to have prefixed icons e.g. "RhUiArrowDownIcon" and "RhMicronArrowDownIcon" and "ArrowDownIcon", see how feasible it is to create a single "ArrowDownIcon" that has an additional "set" property to control which source path data is rendered.
Should be possible in the writeIcons script before exporting everything, just need to be careful in mapping everything and ensure that icons are named consistently (no arrow-down and down-arrow icons because they would be mapped differently). Need to determine a fallback order for sets (pf/fa > rh-ui > rh-microns > etc?) and default set. Need to figure out documentation because not all icons will have support or data for all sets, and icons may have different default sets.
Jira Issue: PF-3403
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 script writeIcons mentionné dans l’issue et suivez la manière dont les noms d’icônes et les chemins source sont associés avant l’exportation. Évaluez la cohérence de l’association des noms, la sélection des ensembles, l’ordre des fallback et le comportement par défaut, puis documentez la faisabilité, les combinaisons d’ensembles non prises en charge et les décisions d’API qui en découlent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react, typescript
- Domaine
- frontend
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100