patternfly / patternfly/react-component-groups
SkeletonTable: add option to hide header skeleton rows
还没有人认领这个 Issue。
- 主要语言
- TypeScript
- 星标
- 10
- 派生
- 40
- 平均合并
- 3 天 22 小时
- 30 天内合并 PR
- 4
描述
Description
SkeletonTable always renders skeleton rows for the table header. There is no prop to suppress header skeletons while keeping the body skeleton rows.
Use Case
In ansible/ansible-ui, our PageLoadingTable intentionally omits header skeletons because the real table header (with column titles, sort controls, etc.) is already rendered above the loading state by the toolbar/layout. Showing skeleton header rows creates a visual mismatch — users see the real header and a duplicate skeleton header simultaneously.
Proposed Solution
Add a boolean prop (e.g., showHeaderSkeleton defaulting to true, or numberOfHeaderRows defaulting to 1 with 0 meaning none) to control whether header skeleton rows are rendered.
Current Workaround
We maintain a custom PageLoadingTable component that renders PF6 Skeleton components for body rows only, without header skeletons.
Jira Issue: PF-4546
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
定位 SkeletonTable 组件及其相关测试,然后了解当前如何渲染 header 和 body skeleton 行。添加一个布尔选项或行数选项,并将现有的 header 行为设为默认值;验证禁用该选项后 body 行仍保持不变,且相关测试通过。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- react, typescript
- 领域
- frontend
- Issue 类型
- 功能
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 活跃
- 描述清晰度
- 基本清楚
- 新手友好度
- 72/100