-
Notifications
You must be signed in to change notification settings - Fork 18
hashに応じてアーカイブページへ遷移する機能を実装 #1263
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
base: main
Are you sure you want to change the base?
Conversation
monaqa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます。助かります。
この PR でなくてもよいのですが、アーカイブページに遷移したときに対応するエントリを一瞬だけ光らせることって簡単に可能だったりしますか?
大量に記事が並んでいるところにページが移るので、どのエントリかすぐ分かるようにしたほうがいいかなと。
|
レビューありがとうございます! 光らせるのいいですね!!帰宅後になりますがやってみます!! |
kawarimidoll
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
途中で作業止まってしまってるので、明日か明後日再開します |
monaqa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
自分から言い出しておいてなんですが、私の指摘は補足的なもので必須ではないと思うので、この PR は一旦マージして別 PR で実施でもよい気がしました!
#1012 (comment)
こちらの対応です。
例:
https://vim-jp.org/ekiden/#article-2025-09-01 がTOPページ内にない場合、https://vim-jp.org/ekiden/archives/#article-2025-09-01 へ遷移
content.jsonに存在しない日付の場合は、遷移しないようにしています。