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

リファクタリング:読み方&アクセント辞書ダイアログの右側パネルを別コンポーネントにする #2234

Closed
3 tasks
jdkfx opened this issue Aug 20, 2024 · 2 comments · Fixed by #2290
Assignees

Comments

@jdkfx
Copy link
Contributor

jdkfx commented Aug 20, 2024

内容

スクリーンショット 2024-08-20 16 04 01

読み方&アクセント辞書ダイアログ(以下、辞書)は、

  • 辞書に登録した単語・読みの一覧など、主に表示・削除に関する処理(左側パネル)
  • 辞書に登録した単語・読みの追加・編集など、主に入力に関する処理(右側パネル)

の二つの要素があります。
この二つの要素をコンポーネントに分割したいです。

Pros 良くなる点

  • 辞書ダイアログのコードが読みやすくなります
  • 今後の修正や改善がやりやすくなります

Cons 悪くなる点

  • 特に思いつくものはありません

実現方法

  • <!-- 右側のpane -->というコメント箇所以下からコンポーネントに分割する

<!-- 右側のpane -->
<div
v-show="wordEditing"
class="col-8 no-wrap text-no-wrap word-editor"
>

VOICEVOXのバージョン

0.20.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

参照:
#2156 (review)

@Hiroshiba
Copy link
Member

とても良いと思います!!ぜひ!!!

@jdkfx
Copy link
Contributor Author

jdkfx commented Aug 20, 2024

これやってます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment