patternfly / patternfly/react-component-groups
SkeletonTable: add option to hide header skeleton rows
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- TypeScript
- Star
- 10
- Fork
- 40
- Merge trung bình
- 3 ngày 22 giờ
- Pull request đã merge (30 ngày)
- 4
Mô tả
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
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Tìm component SkeletonTable và mọi test liên quan, sau đó đọc cách các hàng skeleton của header và body hiện đang được render. Thêm một tùy chọn boolean hoặc tùy chọn số lượng hàng, với hành vi header hiện tại làm mặc định, và xác minh rằng việc tắt tùy chọn này vẫn giữ nguyên các hàng body và các test liên quan đều đạt.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- react, typescript
- Lĩnh vực
- frontend
- Loại issue
- Tính năng
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Sôi nổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 72/100