palantir/blueprint

Hotkey should show "option" text on macOS instead of "alt"

Open

#4959 aperta il 7 ott 2021

Vedi su GitHub
 (0 commenti) (3 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
P2Package: coreType: bughelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

Environment

  • Package version(s): blueprintjs core v3.51.0
  • Browser and OS versions: All browsers; macOS

Feature request

The text for Hotkey when combo has the word "option" should display "option" as the associated text on macOS instead of "alt". This is similar to how "mod" is "cmd" or "ctrl" depending on the operating system.

Examples

<Hotkey
    global
    combo="option"
    label="Test"
    preventDefault
    onKeyDown={() => console.log('test')}
/>

Currently, it automatically renames "option" to "alt" when passed into combo:

Should be:

Guida contributor

Direzione di ricerca
Indaga la logica di normalizzazione dei tasti del componente Hotkey, probabilmente nei file sorgente sotto packages/core/src/components/hotkey. Trova dove 'mod' viene mappato a 'cmd' o 'ctrl' in base al sistema operativo. Replica quel pattern per mappare 'option' in modo che visualizzi 'option' su macOS e 'alt' su altre piattaforme. Controlla i test esistenti per il componente per assicurarti che la modifica sia coperta e non rompa il comportamento esistente.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75