Skip to content

修复无封面文章头部、阅读提示与全站订阅区宽度 - #91

Merged
13304930782 merged 3 commits into
mainfrom
codex/article-no-cover-resume
Oct 13, 2026
Merged

13304930782 merged 3 commits into
mainfrom
codex/article-no-cover-resume

Conversation

@13304930782

Copy link
Copy Markdown
Owner

无封面文章此前仍使用通栏深色色带,并预留封面探出的空间;首页订阅区还被正文宽度限制。现在无封面文章使用主题底色的居中标题区,所有公共订阅区统一采用站点容器宽度。

恢复阅读提示移至屏幕底部,支持暂停计时、继续阅读及本次访问关闭记忆;桌面目录长标题最多显示两行,保留完整标题提示。有封面文章的取色、封面卡片和胶囊样式保持现有表现。

验证:npm run typecheck、npm test(81/81)、npm run build 通过;1440、1024、390 三个宽度及亮暗主题完成本地验收。有封面头部六组改前/改后截图像素一致;阅读提示的进度边界、8秒计时、弹层隐藏、同步错误互斥及重新连接均通过检查和独立复核。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T14:55:21.316079Z 05db4c0 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 05db4c09b2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +23 to +26
useEffect(()=>{
if(!visible||hovered||focused||leaving)return;
const started=Date.now(),timer=setTimeout(dismiss,remaining.current);
return()=>{clearTimeout(timer);remaining.current=Math.max(0,remaining.current-(Date.now()-started));};

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep the resume action reachable before auto-dismissal

For keyboard-only readers, this timer starts immediately while the portaled toast receives no focus and pauses only after one of its controls has already been reached. Since the portal is appended after the application’s existing navigation and article controls, the toast can disappear after eight seconds before the reader can tab to “继续阅读”; the same countdown also runs while the tab is backgrounded or CSS hides the toast behind a dialog/search panel. Keep the action available until explicit dismissal, or pause the timeout whenever it is not perceivable or keyboard-reachable.

AGENTS.md reference: AGENTS.md:L31-L32

Useful? React with 👍 / 👎.

@13304930782
13304930782 merged commit 6e60c13 into main Oct 13, 2026
4 checks passed
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.

1 participant