Skip to content

fix article titles and simplify listings - #23

Merged
LjyYano merged 1 commit into
masterfrom
codex/fix-article-titles
Aug 16, 2026
Merged

LjyYano merged 1 commit into
masterfrom
codex/fix-article-titles

Conversation

@LjyYano

@LjyYano LjyYano commented Aug 16, 2026

Copy link
Copy Markdown
Owner

What changed

  • disable automatic title inference from the first Markdown heading
  • keep the existing filename fallback for all article titles
  • remove the decorative hero from the archive page
  • remove the decorative hero from all six category pages
  • keep filters, article counts, and article lists intact

Root cause

GitHub Pages loads jekyll-titles-from-headings, which populated page.title from the first heading when front matter had no title. Many articles begin with # 前言, so their filenames were incorrectly replaced by 前言 in lists and article pages.

Validation

  • Jekyll YAML configuration parses and disables titles_from_headings
  • Liquid 4 strict parsing passes for the archive and category templates
  • git diff --check passes

@LjyYano
LjyYano marked this pull request as ready for review August 16, 2026 11:27
@LjyYano
LjyYano merged commit 96dd7a4 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: 6c1d6dda9f

ℹ️ 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 archive.html
</header>

<section class="archive-content">
<section class="archive-content" 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 Restore a visible top-level heading

After removing the hero, /archive/ and all pages using the category layout no longer render an <h1>; the archive outline starts at its year <h2> elements and category pages start at article <h2> elements. This removes the visible page identity and the top-level landmark used by heading navigation, especially for users opening a listing directly, so retain a compact page heading when simplifying the decorative hero.

Useful? React with 👍 / 👎.

This branch had an error being deployed

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