palantir/blueprint

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

Open

#4,959 opened on 2021年10月7日

GitHub で見る
 (0 comments) (3 reactions) (0 assignees)TypeScript (20,263 stars) (2,167 forks)batch import
P2Package: coreType: bughelp wanted

説明

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:

コントリビューターガイド

技術スタック
typescriptreact
領域
frontend
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
ReactTypeScriptHotkey component structure
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65
調査方針
Investigate the Hotkey component's key normalization logic, likely in the source files under packages/core/src/components/hotkey. Find where 'mod' is mapped to 'cmd' or 'ctrl' based on OS. Replicate that pattern to map 'option' to display 'option' on macOS and 'alt' on other platforms. Check existing tests for the component to ensure the change is covered and does not break existing behavior.