Skip to content

simplify blog homepage - #22

Merged
LjyYano merged 1 commit into
masterfrom
codex/simplify-homepage
Aug 16, 2026
Merged

LjyYano merged 1 commit into
masterfrom
codex/simplify-homepage

Conversation

@LjyYano

@LjyYano LjyYano commented Aug 16, 2026

Copy link
Copy Markdown
Owner

What changed

  • remove the full decorative homepage hero, including the title, monogram, actions, and statistics
  • remove the category section heading and explanatory copy
  • keep the category cards and recent article list as the homepage entry point
  • reduce the top spacing before the category cards

Why

The removed sections are decorative and not needed for the blog's primary reading workflow.

Validation

  • Liquid 4 strict template parsing passes
  • removed copy no longer appears in index.html
  • git diff --check passes

@LjyYano
LjyYano marked this pull request as ready for review August 16, 2026 10:55
@LjyYano
LjyYano merged commit 9c174b7 into master Aug 16, 2026
1 of 2 checks passed

@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: e9c9bf534e

ℹ️ 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 thread index.html
<p>知识并不生长在孤岛上。技术、故事和生活经验,总会在某处彼此照亮。</p>
</header>

<section class="category-section wrap" aria-label="文章分类">

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 Retain a level-one heading on the homepage

Removing both the hero and category headers leaves the rendered homepage with no <h1> at all, and its first headings are the category-card <h3> elements before the later <h2>. For users navigating by screen-reader heading shortcuts, this produces an incomplete and out-of-order document outline; the section's aria-label names the region but does not provide a page-level heading. Keep a concise heading, potentially visually hidden, and re-level the category headings beneath it.

Useful? React with 👍 / 👎.

This branch had an error being deployed

1 failed deployment
github-pages — e9c9bf53 Deployed Aug 16, 2026 by LjyYano via deploy #4
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