Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KeyboardEvent.KeyCode を削除したことによる不要コードの削除 #129

Open
sapphimoor opened this issue Mar 9, 2022 · 1 comment
Labels
very very easy 超簡単.github に慣れるために新入生にやってもらうのが良き.

Comments

@sapphimoor
Copy link
Contributor

元は

if (e.shiftKey || e.ctrlKey || e.altKey || e.metaKey) {

e.keyCode === keyShift のようにしていたが、変更のため定数が不要となった。
そこで
const keyShift = 16;
const keyCtr = 17;
const keyAlt = 18;
/** Windos の場合は windows button */
const keyMetaL = 91;

の行は全て削除して良い気がしている。

@sano-jin sano-jin added the very very easy 超簡単.github に慣れるために新入生にやってもらうのが良き. label Mar 10, 2022
@sano-jin
Copy link
Contributor

補足:

npm run lint

すると不要変数がわかる.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
very very easy 超簡単.github に慣れるために新入生にやってもらうのが良き.
Projects
None yet
Development

No branches or pull requests

2 participants