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

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

Open
3 of 17 tasks
Hiroshiba opened this issue Feb 18, 2024 · 6 comments
Open
3 of 17 tasks

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

Hiroshiba opened this issue Feb 18, 2024 · 6 comments
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 18, 2024

内容

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

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

リスト

@weweweok
Copy link
Contributor

weweweok commented Feb 25, 2024

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

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

@weweweok
Copy link
Contributor

weweweok commented Mar 8, 2024

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

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
Copy link
Member Author

Hiroshiba commented Mar 8, 2024

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

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

@Hiroshiba Hiroshiba pinned this issue Mar 22, 2024
@Hiroshiba
Copy link
Member Author

Hiroshiba commented Mar 23, 2024

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

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

@weweweok
Copy link
Contributor

weweweok commented Apr 6, 2024

@z80oolong

This comment was marked as spam.

@Hiroshiba Hiroshiba changed the title リファクタリングできそうな箇所集 good first issue:リファクタリングできそうな箇所集 Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
Development

No branches or pull requests

3 participants