ShortcutGrid: Add internationalization (i18n) support for Shortcut grid component
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 58/100
- Type d'issue
- Fonctionnalité
- Clarté
- Clairement spécifiée
- Activité
- Calme
- Stack technique
- react, typescript
- Domaine
- frontend, internationalization
Piste de recherche
Commencez dans Shortcut.tsx, en particulier par les labels codés en dur existants autour des lignes 38–54 et par l’interface ShortcutProps. Ajoutez des labels facultatifs qui conservent les valeurs par défaut actuelles en anglais, puis vérifiez que chaque prop booléenne d’action de la souris affiche son label personnalisable, tout en préservant l’utilisation existante de ShortcutGrid.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Describe the problem
The Shortcut component (used by ShortcutGrid) currently has hardcoded English strings for mouse action labels with no way to internationalize them for applications that need to support multiple languages.
Current hardcoded strings
-
"Hover"(line 38 of Shortcut.tsx) -
"Click"(line 45) -
"Right click"(line 48) -
"Drag"(line 51) -
"Drag + Drop"(line 54)
These are rendered inside Label components alongside a MouseIcon when the corresponding boolean props (hover, click, rightClick, drag, dragAndDrop) are set to true.
Proposed solution
Add optional label props for each mouse action to allow customization of all user-facing strings:
export interface ShortcutProps {
// ... existing props
/** Custom label for "Hover" mouse action */
hoverLabel?: string;
/** Custom label for "Click" mouse action */
clickLabel?: string;
/** Custom label for "Right click" mouse action */
rightClickLabel?: string;
/** Custom label for "Drag" mouse action */
dragLabel?: string;
/** Custom label for "Drag + Drop" mouse action */
dragAndDropLabel?: string;
}
These would default to the current English strings for backwards compatibility.
Example usage
import { useTranslation } from 'react-i18next';
const MyComponent = () => {
const { t } = useTranslation();
return (
<ShortcutGrid
shortcuts={[
{
keys: ['ctrl'],
drag: true,
dragLabel: t('Drag'),
description: t('Move object'),
},
{
keys: [],
rightClick: true,
rightClickLabel: t('Right click'),
description: t('Open context menu'),
},
]}
/>
);
};
Impact
Applications using Shortcut or ShortcutGrid that need to support non-English languages currently have to either:
-
Accept English-only mouse action labels
-
Avoid the boolean mouse action props entirely and lose the
MouseIcon+Labelbadge rendering -
Build their own shortcut component from scratch
Related
This would align ShortcutGrid with other PatternFly components that support i18n.
Jira Issue: PF-3867
- Langage dominant
- TypeScript
- Étoiles
- 10
- Forks
- 40
- Merge moyen
- 12 min
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire 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.
Autres issues de patternfly/react-component-groups
-
SkeletonTable: add option to hide header skeleton rowsPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
patternfly/react-component-groups#968 · 1 commentaire ·
-
ColumnManagementModal: add the ability to have sticky columnsPeut-être pris @logonoff l’a pris il y a 6 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Tearsheet: Escape should only close the active topmost layerPeut-être pris @GAUNSD l’a pris aujourd’hui. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
patternfly/react-component-groups#989 · 2 commentaires · 1 personne assignée ·
-
Tearsheet layout updates: Masthead coverage and inset token alignmentPeut-être pris @GAUNSD l’a pris il y a 13 jours. OuvertePF Team
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
patternfly/react-component-groups#985 · 1 commentaire · 1 personne assignée ·
-
Pull in Modal animation support for TearsheetsPeut-être pris @thatblindgeye l’a pris il y a 27 jours. OuvertePF Team
patternfly/react-component-groups#979 · 1 personne assignée ·
Toutes les issues de patternfly/react-component-groups
Issues similaires
-
priority:high
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
immich-app/immich#32426 ·
Les mainteneurs répondent en général sous 1 jour
-
documentation good first issue hacktoberfest help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
libredb/libredb-studio#1756 ·
Les mainteneurs répondent en général sous 1 jour
-
state:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour