一部のキーを押すと全押し状態になる問題の修正

[suzme]

🔨 変更内容 / Details of Changes

• プレイ中のキー押下処理で_keyCode === ''のとき処理を行わないようにします

🔖 関連Issue, 変更理由 / Related Issues, Reason for Changes

• 無変換など一部の特殊キーを押したとき全押し状態になってしまいます
• キーコードが空文字で、無効化された代替キーに反応してしまうようです

📷 スクリーンショット / Screenshot

📝 その他コメント / Other Comments

• 環境依存かもしれません

[cwtickle]

PRありがとうございます。
調べてみたところ、音量変更ボタンはkeyboardEvent.keyには値がありますが
keyboardEvent.codeには値がありませんでした。
おそらく、こういった場合に問題が起きていると思われます。

キー操作では現状、keyboardEvent.codeを採用しています。
keyboardEvent.codeを受け取った場合、g_kCdNの値から番号(従来のキーコード相当)を取得し、
g_kCdに対応する値をキーコンフィグ画面へ表示します。
下記の場合、keyboardEvent.codeでBackspaceを受け取ると8番が返され、「BS」と表示します。
このとき、上記のように空を受け取った場合は0番を返し、「- - -」と表示していました。

g_kCdN[8] = `Backspace`;
g_kCdN[0] = ``;

g_kCd[8] = `BS`;
g_kCd[0] = `- - -`;

0番はオート/無効値で使用するため、本来割り当ててはいけない番号です。
下記のように、空を割り当てる番号を0から1に変えれば問題は起こらなそうですが、どうでしょうか。
（手元では問題が無いことを確認しています）

• danoni_constants.js 側の設定

g_kCdN[0] = `XXX`;  // 割り当てできないように任意の文字で置き換え
g_kCdN[1] = ``;        // 未使用番号をcode=空に割り当て

g_kCd[0] = `- - -`;    // 無効用表示
g_kCd[1] = `Volume`;    // 音量などの表示として使用（表示対象ではないが、念のため設定）

• danoni_main.js 側の設定（5,399行目）

// 先に設定した番号：1を割り当て禁止
const disabledKeys = [1, 229, 240, 242, 243, 244, 91, 29, 28, 27, g_prevKey];

上記を対策することで、メイン画面上でキーブロックする必要はなくなると思います。

[codeclimate]

Code Climate has analyzed commit bcf33e7 and detected 0 issues on this pull request.

View more on Code Climate.

[suzme]

danoni_constants.jsで設定するように修正しました。
ボリュームとは限らないので、表示するならUnknownとかがいいかなと。

割り当て禁止にはしなくても問題は無さそうですがどうでしょうか。

[cwtickle]

修正ありがとうございます。
結果的に全部直して頂くことになってしまいましたね・・・＼(^o^)／

disabledKeyに追加するのは必須で無くても良いと思います。
この内容でマージしますー。