Skip to content

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

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 titlealign to right side 与代码一致 —— 标题行是 flex flex-wrap(:211),右槽 ml-auto(:235),窄屏确实会换行。只有 Spacing 这一条是假的。

建议处置

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

参考位置

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

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions