Skip to content

feat: 階層構造の可視化のため、サイドバーを追加 - #30

Merged
Tatsu723 merged 3 commits into
mainfrom
Sidebar
Aug 28, 2026
Merged

feat: 階層構造の可視化のため、サイドバーを追加#30
Tatsu723 merged 3 commits into
mainfrom
Sidebar

Conversation

@raito-ctrl

Copy link
Copy Markdown
Collaborator

feat: 階層構造の可視化のため、サイドバーを追加

@raito-ctrl
raito-ctrl requested review from Tatsu723 and tknkaa August 21, 2026 02:59

@tknkaa tknkaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

全体的にはよさそう!
細かいところだけコメントした!

Comment thread components/Sidebar.tsx

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

コンポーネント関数をデフォルトエクスポートするファイルの名前はパスカルケースにしよう!
sidebar.tsx → Sidebar.tsx

Comment thread components/Sidebar.tsx
{/* 言語名のボタン(クリックで言語ページへの遷移とセクション表示のトグル開閉を同時に行う) */}
<Link
href={`/learn/${lang.languageSlug}`}
onClick={() => toggleLanguage(lang.languageSlug)}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

言語名を押しても toggle できるようにするのは割とよくある UI だと思うけど、それなら「My単語帳」と「学習教材」も同じ実装にしたい!

@Tatsu723
Tatsu723 temporarily deployed to Sidebar - vocabook PR #30 August 28, 2026 04:34 — with Render Destroyed
@Tatsu723
Tatsu723 merged commit 949c123 into main Aug 28, 2026
2 checks passed
@Tatsu723
Tatsu723 deleted the Sidebar branch August 28, 2026 05:35
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