Repository navigation
ランキングの NEW の窓を30日から14日に絞る (#3408) - #3410
Merged
Merged
Conversation
トレンドは直近30日の PV で並ぶので、NEW の窓が30日だと集計期間と重なり、 行の半分以上に付いて新顔と定番が見分けられない。窓は NEW_DAYS の1箇所が持つ。 Tech Cast のコメントにあった記事の NEW との日数比較は成り立たなくなるので外した。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
記事の NEW との比較を外したときに 15日の理由まで消えていたので、「今週出た回」の合図であることだけを書く。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
変更
scripts/lib/post_list.js）NEW_DAYSの1箇所だけが持つ。JSDoc の「公開30日以内」もこの名前で指すようにしたscripts/techcast_latest.jsのコメントから、記事の NEW が30日であることを前提にした比較を外した。窓を14日にすると Tech Cast の15日より短くなり、説明が成り立たなくなるCLAUDE.md・スタイルガイド・部品ギャラリーには、NEW の日数を名乗っている箇所が無かった。
理由
トレンドは直近30日の PV で並ぶので、NEW の窓が30日だと集計期間と重なる。行の半分以上に付く印は合図として働かず、隣の年月とも同じことを言っている。14日に絞ると「今週・先週に出た記事」だけが NEW を持ち、トレンドの中でも新顔と定番が見分けられる。
NEW の行数（2026-10-30 のトップ）
変更後は生成した
public/index.htmlの NEW を数えた値で、変更前は同じ行の記事の日付から30日の条件で数えた値。Closes #3408
🤖 Generated with Claude Code