search

VOICEVOX / voicevox

無料で使える中品質なテキスト読み上げソフトウェア、VOICEVOXのエディター
https://voicevox.hiroshiba.jp/
Other
2.48k stars 300 forks source link

good first issue：リファクタリングできそうな箇所集 #1854

Open Hiroshiba opened 7 months ago

Hiroshiba commented 7 months ago

内容

リファクタリングできそうだけれども後回しになっているところを列挙しています！ 追加が必要であれば自由にeditしてください！ edit権限がない場合はコメントをお願いします 🙏

たぶん難易度や規模感やリファクタリング方針などが書かれていた方が取り組みやすそうなのですが、一旦適当に難易度だけ書いています。

リスト

weweweok commented 7 months ago

SettingDialog.vueでコピペされてる部分をv-forにする

こちら取り組んでみようと思います。

weweweok commented 7 months ago

申し訳ありません...!!!　諦めます🙇‍♂

SettingDialog.vueでコピペされてる部分をv-forにする

このあたりをv-forでシンプルにできれば、

                   <template #CONTINUOUSLY>
                    <q-tooltip :delay="500">
                      現在の再生位置を真ん中に表示します。
                    </q-tooltip>
                  </template>
                  <template #PAGE>
                    <q-tooltip :delay="500">
                      現在の再生位置が表示範囲外にある場合にスクロールします。
                    </q-tooltip>
                  </template>
                  <template #OFF>
                    <q-tooltip :delay="500">
                      自動でスクロールしません。
                    </q-tooltip>
                  </template>

このあたりはすべてv-forで回せそうです。 image

実際、v-forでここまでは書くことができました。 (そもそも、 SettingDialog.vueでコピペされてる部分をv-forにするの意味を理解できているか不安ですが。。。) image

Hiroshiba commented 7 months ago

@weweweok コメントありがとうございます！！ 画像見た感じ、惜しいところまで行けてそうな感じがしました！ 元々のタスク内容と取り組んでくださった内容の方向性は合っていたと思います！ 取り組んでくださってありがとうございました！！

もしよければ他のにもチャレンジしてみてください！

Hiroshiba commented 6 months ago

SettingDialog.vueでコピペされてる部分をv-forにする

こちら自分も挑戦してみようと思います！

weweweok commented 6 months ago