patternfly / patternfly/react-component-groups

SkeletonTable: add option to hide header skeleton rows

未关闭 适合新手
#968 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 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

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

定位 SkeletonTable 组件及其相关测试,然后了解当前如何渲染 header 和 body skeleton 行。添加一个布尔选项或行数选项,并将现有的 header 行为设为默认值;验证禁用该选项后 body 行仍保持不变,且相关测试通过。

由索引模型根据 Issue 内容生成。

评估

技术栈
react, typescript
领域
frontend
Issue 类型
功能
难度
2/5
预计耗时
1-3 小时
活跃度
活跃
描述清晰度
基本清楚
新手友好度
72/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。