Skip to content

cwn-jaへのリンクを追加#17

Merged
mt-caret merged 3 commits into
mainfrom
feature/add-cwn-ja-link
Apr 17, 2026
Merged

cwn-jaへのリンクを追加#17
mt-caret merged 3 commits into
mainfrom
feature/add-cwn-ja-link

Conversation

@mt-caret
Copy link
Copy Markdown
Contributor

@mt-caret mt-caret commented Apr 17, 2026

cwn-jaへのリンクを追加した上で、右上のリンク一覧をハンバーガーメニューにまとめてみました。

image image

@mt-caret mt-caret requested a review from lmdexpr April 17, 2026 04:07
@lmdexpr
Copy link
Copy Markdown
Member

lmdexpr commented Apr 17, 2026

@mt-caret

ありがとうございます!

「Community & Links」への追加なんですが、問題ない一方で、現在は他の要素が外部へのリンクだけになっているのが気になりました。
cwn-ja はレポジトリこそ違うものの、ocaml.jp 配下なので「イベント一覧」と同じ扱いにしても良い気がします。

この辺り、何かお考えあればコメントいただけると助かります。
個人的には上記の通りですが、イベント一覧の導線がすごく分かりやすい訳ではないので、今回の修正でも良いとは思います。
あるいは、「Community & Links」とは別に ocaml.jp 内の導線を似たようなカードで作っても良いかなと思いました。

@mt-caret
Copy link
Copy Markdown
Contributor Author

おっしゃる通りですね、別な区分けを作った上で右上の導線をハンバーガーメニューにまとめてみましたが、どうでしょう?

Copy link
Copy Markdown
Member

@lmdexpr lmdexpr left a comment

Choose a reason for hiding this comment

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

ありがとうございます!

(最近 rescript も使っているので html 側の色々で利用してみてもいいかもなと思いました)

@mt-caret mt-caret force-pushed the feature/add-cwn-ja-link branch from 3fbcb87 to 706f075 Compare April 17, 2026 07:13
@mt-caret mt-caret merged commit 0c0f4e0 into main Apr 17, 2026
1 check passed
@mt-caret mt-caret deleted the feature/add-cwn-ja-link branch April 17, 2026 07:14
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