Skip to content

fix: standardize blog cover presentation - #69

Merged
zhourrr merged 3 commits into
zvec-ai:mainfrom
feihongxu0824:codex/fix-blog-cover-crop
Oct 21, 2026
Merged

zhourrr merged 3 commits into
zvec-ai:mainfrom
feihongxu0824:codex/fix-blog-cover-crop

Conversation

@feihongxu0824

Copy link
Copy Markdown
Contributor

Summary

  • render featured and article-card covers consistently without blank bands or per-post fit overrides
  • normalize legacy cover assets and add dark-mode covers for the RaBitQ and multi-platform posts
  • add a 16:9 cover specification plus automated validation for local paths, dimensions, and aspect ratio
  • run the cover validator automatically during production builds

Validation

  • npm run blog:covers:check
  • npm run types:check
  • npm run build

@github-actions

github-actions Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

☁️ Cloudflare Pages preview: https://d17c8ea1.zvec-web.pages.dev

Comment thread CONTRIBUTING.md

@zhourrr zhourrr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

基于提交 8677648 做了代码检查、本地生产构建和浏览器验证。统一封面比例、补充暗色资源和接入构建检查的方向合理;现有 23 个封面资源校验、npm run types:check 和 npm run build 均通过。

下面提交 3 条行内意见,分别说明置顶封面的响应式裁切、YAML 行内注释导致的构建误报,以及必填封面的校验遗漏。每条都附了触发条件、复现结果和修改建议,建议先修复前两项,再合并。

Comment thread app/global.css
Comment thread scripts/validate-blog-covers.mjs Outdated
Comment thread scripts/validate-blog-covers.mjs Outdated
@zhourrr
zhourrr merged commit 3f9450d into zvec-ai:main Oct 21, 2026
1 check passed
@feihongxu0824
feihongxu0824 deleted the codex/fix-blog-cover-crop branch October 21, 2026 11:43
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.

2 participants