fix: generalize site footer content - #117
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Responsive design follow-up:
Validation completed at 375px, 768px, and 1440px with no horizontal overflow. |
|
已按项目样式约定继续优化:删除 Footer.module.less(92 行),改由 Bootstrap 5.3 栅格和工具类承担布局、间距、分隔、对齐及响应式展示。已验证 375/768/1440px:无横向溢出;平板两列等分;桌面三列宽度均为 437.33px;开源图书馆仍保留 Catalog / How to Borrow 和原描述。pnpm test 与 CodeQL 均通过。Build-and-Deploy 仍因 Vercel Team fair-use 限额失败。提交:31786fa。 |
Co-authored-by: South Drifter <shiy2008@gmail.com>
Co-authored-by: South Drifter <shiy2008@gmail.com>
Summary
_app.tsxbased on the current routeContext
This follows the reviewer feedback in #116: keep one Footer component and pass route-specific content into it instead of maintaining two independent footer implementations.
Validation
pnpm testpassedgit diff --checkpassedCI=true NEXT_TELEMETRY_DISABLED=1 pnpm exec next build --webpack/,/article/about,/open-library, and/open-library/booksin a local browserClose #115