MobileHeaderBarをHeaderBarと統合 #44
Conversation
sevenc-nanashi
requested review from
Hiroshiba
and removed request for
a team
|
こんな感じになりました。 |
具体的にどんな感じです？（ツールバーの一番左がMenuBar代わり） |
|
@sevenc-nanashi ニュアンスはこんなかんじかなと！ メニューバーを開いたり、設定を開いたりは頻度が低いから上。 （あくまで考え方の１つって感じかなと） |
|
設定とかはプロジェクト一覧（Discordに置いてあるイメージ図）とかのメニューとして出そうかな～って思ってます。 |
|
かといってメニュー開くボタンが他のボタンほど使われるのか微妙な気がするんですよね…。 |
src/components/HeaderBar.vue
Outdated
| @@ -1,5 +1,22 @@ | |||
| <template> | |||
| <q-header class="q-py-sm"> | |||
| <q-toolbar v-if="isMobile" class="bg-primary text-white"> | |||
There was a problem hiding this comment.
ここ、コンテナとプレゼンター分けて、モバイル用のプレゼンター入れた方がいい気がしますね〜
リファクタリング箇所がわかってきた。
src/components/MobileMenu.vue
Outdated
There was a problem hiding this comment.
デフォルトスタイル選択とかってどこにいくことになりそうでしょう
There was a problem hiding this comment.
なるほどです。
設定への遷移をプロジェクト選択画面に持たせる場合、３遷移になって流石に遠いかもですねぇ。
（そもそもスマホアプリでそんな細かい設定があるものなのか知らないですが。。）
|
VITE_TARGETの分岐はほとんどないので、
で動くと思います。 |
| // 1フレーム遅らせることで、真っ白の画面が表示されるのを防ぐ。 | ||
| requestAnimationFrame(() => { | ||
| SplashScreen.hide(); | ||
| }); |
内容
タイトル通り。
関連 Issue
（なし）
スクリーンショット・動画など
（なし）
その他
（なし）