Skip to content
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
This repository was archived by the owner on Nov 4, 2026. It is now read-only.

docs: page-header.mdx 的 Responsive Behavior 仍写「Spacing: Adjusts padding for different screen sizes」—— 与 #3786 修掉的 pb-8 on desktop 是同一个假断言 #3902

Closed

Description

@yinlianghui

发现于 #3786 的实现(PR:claude/issue-3786-3787-pageheader-docs-demo),不在该 PR 处理 —— 那一单的判定面被明确钉在「Styling → Container 三条」,这一条在另一个小节(Responsive Behavior),是 #3786 正文没有列举的第四份副本。

现象

#3786 修掉了 Styling → Container 里的 pb-4 on mobile / **pb-8 on desktop**(实际 pb-4`,无任何断点变体)。但同一文件末尾的 Responsive Behavior 小节还留着同一个断言的另一种说法:

content/docs/layout/page-header.mdx:225-229

## Responsive Behavior

- **Mobile**: Actions stack below title if needed
- **Desktop**: Actions align to right side
- **Spacing**: Adjusts padding for different screen sizes

packages/layout/src/PageHeader.tsx:210 的根 class 是 flex flex-col gap-3 pb-4 border-b —— padding 不随屏幕尺寸变化,一个断点变体都没有。整个组件里唯一的响应式类是标题的 md:text-3xl(:231),那是字号,不是 padding。

为什么值得记一笔

#3786 修完后这一页是自相矛盾的:Container 小节说「pb-4 at every breakpoint — there is no responsive variant」,四行之后的 Responsive Behavior 说 padding 会按屏幕尺寸调整。读者(覆盖间距的作者)看到哪一条取决于翻到哪一节。这正是 #3786 治的那个病,只是换了个小节。

顺带核过、没有问题的两条:Actions stack below title 与 align to right side 与代码一致 —— 标题行是 flex flex-wrap(:211),右槽 ml-auto(:235),窄屏确实会换行。只有 Spacing 这一条是假的。

建议处置

删掉 Spacing 那一行,或改成如实的说法(唯一的响应式行为是标题 text-2xl → md:text-3xl)。不到 2 行。

参考位置

  • content/docs/layout/page-header.mdx:225-229(Responsive Behavior)
  • packages/layout/src/PageHeader.tsx:210、:231

关联:#3786(同一假断言的另一份副本,发现于此)、#3787

Activity

  1. os-zhuang commented on Aug 9, 2026

    @os-zhuang
    Contributor

    Triage: pm:queue.

    本评论来自分诊座位 Routine(#5474 试点),不构成认领。


    Generated by Claude Code

  2. self-assigned this
    on Aug 9, 2026
  3. yinlianghui commented on Aug 9, 2026

    @yinlianghui
    ContributorAuthor

    🔒 分诊 + 认领(objectui shard PM 亲跑分诊轮,session session_01GTRjn8xBqp75dk7kFupVRt)

    定级:concrete docs —— PR3905 dev 立卡:Responsive Behavior 的「Spacing: Adjusts padding for different screen sizes」(:229 一带)是 #3786 已修假断言的第四份副本,PR3905 落地后该页自相矛盾;同小节另两条已核无误。<2 行修正。


    Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions