Skip to content

ランキングの NEW の窓を30日から14日に絞る (#3408) #3410

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.

Sign up for GitHub

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

Merged
ma91n merged 2 commits into from
Oct 30, 2026
Merged

ランキングの NEW の窓を30日から14日に絞る (#3408) #3410

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3a8a15a
ランキングの NEW の窓を30日から14日に絞る (#3408)
ma91n Oct 30, 2026
dd78df8
Tech Cast の新着の窓に理由を1行で戻す (#3408)
ma91n Oct 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 8 additions & 5 deletions scripts/lib/post_list.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,21 @@

const { getSNSCnt } = require('./sns');

// 公開から30日以内かどうか。
//
// **NEW を出すのはランキングだけ**（呼び出し側が withNew で渡す、#2788）。
// 関連記事は関連度順、参照している記事は張られたリンクの記録で、どちらも
// 新しさが並びの理由ではない。実測では参照している記事の92%が NEW
// （12ページ中10ページが全行 NEW）で、しかも並びが日付降順なので二重だった。
// 関連記事は逆に全期間で0.5%しか出ず、出るのは同じ連載の同時期の記事。
// ランキングは PV 順で新しさと無関係なので、「まだ読んでいないかもしれない
// 新顔」の合図として働く（74行中14行）
// 新顔」の合図として働く
//
// 窓はトレンドの集計期間より短くする。同じ長さだとトレンドの
// 行の半分に付き、新顔と定番が見分けられない (#3408)
const NEW_DAYS = 14;

const isRecent = (date) => {
const threshold = new Date();
threshold.setDate(threshold.getDate() - 30);
threshold.setDate(threshold.getDate() - NEW_DAYS);
return threshold.toISOString() <= date.toISOString();
};

Expand All @@ -30,7 +33,7 @@ const isRecent = (date) => {
* @param {string} [opts.titleAttr] a の title 属性。省略時は lede
* @param {boolean} [opts.withThumb] タイトルの左に小さいサムネを添える (#2230)
* @param {{html: string, className: string}} [opts.rankMark] 行頭に出す順位の丸 (#2249)
* @param {boolean} [opts.withNew] 公開30日以内なら NEW を出す (#2788)
* @param {boolean} [opts.withNew] 公開から NEW_DAYS 日以内なら NEW を出す (#2788)
*/
const postListItem = (
ctx,
Expand Down
3 changes: 1 addition & 2 deletions scripts/techcast_latest.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,7 @@ let feedItems = [];
});
})();

// 公開から15日以内を新着として扱う。記事一覧の NEW（scripts/lib/post_list.js）が
// 30日なのに対して短いのは、こちらが「今週出た回」の合図で、記事の新顔とは役が違うため
// 公開から15日以内を新着として扱う。「今週出た回」の合図
function isNewEpisode(isoDate) {
const threshold = new Date();
threshold.setDate(threshold.getDate() - 15);
Expand Down
Loading