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

feat: ヘッダーとサイドバー・サイドドロワーを実装 #16

Merged
merged 7 commits into from
Jan 1, 2025

Conversation

reiroop
Copy link
Collaborator

@reiroop reiroop commented Dec 30, 2024

テンプレートの不要なコンポーネントとcss を削除
モーダルを実装
ナビゲーションメニューを実装
サイドバー・サイドドロワーを実装
サイドバー・サイドドロワーにナビゲーションメニューを載せる
postcss.config.js を修正
Heroicons を依存関係に追加


意見が欲しいところ

  • よりよいディレクトリ構造
  • よりよい実装
  • よりよい命名

@reiroop reiroop force-pushed the feat/header-and-sidebar branch 2 times, most recently from c4efbad to 89a9dfb Compare December 30, 2024 16:39
サイドバー・サイドドロワーにはナビゲーションメニューを実装
postcss.config.js を修正し
Heroicons を依存関係に追加
テンプレートの不要なコンポーネントとcss を削除
@reiroop reiroop force-pushed the feat/header-and-sidebar branch from ca5447e to f7760fc Compare December 30, 2024 16:44
@Pugma Pugma self-requested a review December 31, 2024 04:01
@Pugma
Copy link

Pugma commented Dec 31, 2024

見た感じ、 PC 画面で表示されるサイドバーでも gap-2 が適用されてそうな気がします
適用されてない部分がもしありましたら、もう少し詳細に教えていただけないでしょうか…?

見つけました
image

@Pugma
Copy link

Pugma commented Dec 31, 2024

色々確かめてみたんですが、そもそもサイドバーとサイドドロワーで表示したいものが同じなのに別の .vue ファイルを作っているのでややこしくなってそうかな〜と思いました
CSS も動的に付け替えることができるので、それを使ってみるともう少し簡素な実装にできそうです

Copy link

@Pugma Pugma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

とりあえずの問題点は見つけたので、ここだけまずは見てもらえればいいかなと思います

src/App.vue Outdated Show resolved Hide resolved
@reiroop reiroop force-pushed the feat/header-and-sidebar branch from 3f9f77e to 9544700 Compare December 31, 2024 16:51
@reiroop reiroop requested review from Pugma and cp-20 December 31, 2024 16:54
Copy link

@Pugma Pugma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます〜
とりあえず気になったのはこの辺りです…!
そんなに優先度が高いものではないので、お時間のあるときにご確認いただければと思います

src/layouts/SideNavigation.vue Show resolved Hide resolved
src/layouts/SideNavigationButton.vue Outdated Show resolved Hide resolved
@reiroop reiroop requested a review from Pugma December 31, 2024 18:07
This was linked to issues Dec 31, 2024
Copy link

@Pugma Pugma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントにも対応くださってありがとうございました
よさそうです!

@reiroop reiroop merged commit 055fbd1 into main Jan 1, 2025
6 checks passed
@reiroop reiroop deleted the feat/header-and-sidebar branch January 1, 2025 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

サイドバーの作成 ヘッダーの作成
3 participants