Skip to content

サイドバーを追加 - #19

Open
raito-ctrl wants to merge 1 commit into
mainfrom
sidebar
Open

サイドバーを追加#19
raito-ctrl wants to merge 1 commit into
mainfrom
sidebar

Conversation

@raito-ctrl

Copy link
Copy Markdown
Collaborator

サイドバーを追加しました。

@Tatsu723

Copy link
Copy Markdown
Contributor

サイドバーの全体としての構成は良い。学習教材の方について、言語→セクションとネストを施している点、及び選択しているものを緑でハイライトするようにしている点も評価できる。
一方、nav-data.tsが冗長であるように見える。言語情報及びセクション情報はcontent.tsとlanguages.tsから得られるはず。重複する内容をコードでベタ打ちしまうのは後々のエラーの原因になりうるので改善したい。

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