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

[refactor] MenuBarを1つにする #1860

Closed
Hiroshiba opened this issue Feb 20, 2024 · 1 comment · Fixed by #1966
Closed

[refactor] MenuBarを1つにする #1860

Hiroshiba opened this issue Feb 20, 2024 · 1 comment · Fixed by #1966
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 20, 2024

内容

ウィンドウの一番上に表示されているメニューバーですが、表示は1つに見えて実はソング・トーク用のメニューバーが2つが用意されています。
これを1つにするタスクです。

Pros 良くなる点

MenuBar内にある、アプリで1回だけ実行されて欲しい処理が実行できるようになる。
(ショートカットキーの登録など)

実現方法

2つのメニューバーはBaseMenuBar.vueを子コンポーネントにしているラッパーの形になっています。
そうではなく、App.vueの直下にBaseMenuBarが1つだけ存在する形を目指すことになると思います。

問題はBaseMenuBarのpropsのfileSubMenuDataですが、とりあえず親(App.vue)から必要なデータを受け取る形が良いかなと。
あとはApp.vueでこんな感じでfileSubMenuDataを渡せばいいかなと!

const fileSubMenuData = computed(() => {
  // editor = "song" もしくは "talk"
  return {talk: talkFileSubMenuData, song: songFileSubMenuData}[editor]
})

Pros 良くなる点

MenuBar内にある、アプリで1回だけ実行されて欲しい処理が実行できるようになる。
(ショートカットキーの登録など)

その他

Vue的な知識が必要になり、ちょっと設計の知識も必要になりますが、結構勉強になると思うのでもしよければ!
(分からないこととか、こういう設計で合ってる?とかあれば聞いてください!)

@sevenc-nanashi
Copy link
Member

やってみようとおもいます。

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

Successfully merging a pull request may close this issue.

2 participants