Skip to content

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

Open
raito-ctrl wants to merge 2 commits into
mainfrom
Sidebar
Open

feat: 階層構造の可視化のため、サイドバーを追加#30
raito-ctrl wants to merge 2 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単語帳」と「学習教材」も同じ実装にしたい!

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.

2 participants