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

リファクタリング:キー割り当て画面で、ショートカット設定ダイアログを別コンポーネントにする #2002

Closed
Hiroshiba opened this issue Apr 19, 2024 · 2 comments · Fixed by #2209
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

ショートカットキー設定画面が若干複雑化しつつあるので、解体してのリファクタリングを募集しています。

ショートカット設定ダイアログは、「キー割り当て」を押したあとショートカットキーを設定しようとすると出てくる↓のダイアログのことです。
image

今はキー割当画面と同じコンポーネント内に記述されていて、かなりややこしくなっています。
これを解体して2つのコンポーネントにしてあげたいです。

実現方法

キー割り当て画面は src/components/Dialog/HotkeySettingDialog.vue にあります。

その中のショートカットキー設定ダイアログはこのあたりです。

分解はvueの知識こそ必要ですが、綺麗に分けられるとは思います。

その他

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク labels Apr 19, 2024
@Hiroshiba Hiroshiba changed the title リファクタリング:ショートカットキー設定画面で、ショートカット設定ダイアログを別コンポーネントにする リファクタリング:キー割当画面で、ショートカット設定ダイアログを別コンポーネントにする Apr 19, 2024
@Hiroshiba Hiroshiba changed the title リファクタリング:キー割当画面で、ショートカット設定ダイアログを別コンポーネントにする リファクタリング:キー割り当て画面で、ショートカット設定ダイアログを別コンポーネントにする Apr 19, 2024
@jdkfx
Copy link
Contributor

jdkfx commented Aug 7, 2024

試しにやってみたらうまくいきそうだったのでこちら取り組みます!

@Hiroshiba
Copy link
Member Author

@jdkfx コメントありがとうございます、ぜひぜひ!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
2 participants