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

縦方向に長いかつこれからもコンテンツの増える可能性の高そうなページを分割 #186

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

syuwa-oisi
Copy link
Collaborator

スクショ貼ろうにもどう貼っていいかが難しいので手元で
http://localhost:1313/hakoniwa/technical-links/
および
http://localhost:1313/hakoniwa/tutorial/
を確認してください。

@mitsut
Copy link
Member

mitsut commented Sep 27, 2024

@syuwa-oisi PRありがとうございます!

  • tutorial
    ちょっと階層が深いような気もしますが、それは別に各紹介ページのところで、上の階層にあがるリンクがほしいです。
    例えば、http://localhost:1313/hakoniwa/introduction-party/2/ の場合に、このページの先頭に戻る がありますが、これを上の階層へのリンク(http://localhost:1313/hakoniwa/introduction-party/)とかにするのはどうでしょうか?
top/
 +-- tutorial
      +-- tutorial-party
      |.   +-- 1
      |.   +-- 2..
      +-- introduction-party
  • technical-links
    こちらはいいと思います!

@syuwa-oisi
Copy link
Collaborator Author

  • tutorial
    ちょっと階層が深いような気もしますが、それは別に各紹介ページのところで、上の階層にあがるリンクがほしいです。
    例えば、http://localhost:1313/hakoniwa/introduction-party/2/ の場合に、このページの先頭に戻る がありますが、これを上の階層へのリンク(http://localhost:1313/hakoniwa/introduction-party/)とかにするのはどうでしょうか?
top/
 +-- tutorial
      +-- tutorial-party
      |.   +-- 1
      |.   +-- 2..
      +-- introduction-party

階層についてはどうしようもないというか、アイデアがないのですが、上の階層へのリンクについては記事内ではなくページネーションのコントローラの一部的な解釈として今行ったPushのような形で良いかなと思うのですがいかがでしょうか?

Copy link
Member

@mitsut mitsut left a comment

Choose a reason for hiding this comment

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

LGTM!
いいと思いますので、PR取り込みます!

@mitsut mitsut merged commit db5b138 into web Oct 2, 2024
2 checks passed
@mitsut mitsut deleted the divide_long_page branch October 2, 2024 05:36
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