improve Table API with regards to Sorting
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- react, typescript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Start by reading the Table component API and its sorting implementation, then find the tests that cover sortable columns. Remove columns.isSortable and add sort.sortableColumns as an array of column names; update the API tests to verify sorting uses those names and that columns, rows, and sort configuration stay aligned.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Presently it is possible to express in Table component API invalid configuration.
It is valid to set columns.isSortable = true and not provide the sort object. We could validate this via a custom Prop validation function, but I think it more elegant to:
- remove the
columns.isSortableparam - add new
sort.sortableColumnsparam holding an array of column names
With this change the user only has to ensure that the column names within params columns, rows, and sort match.
- Ngôn ngữ chính
- TypeScript
- Star
- 23
- Fork
- 10
- Merge trung bình
- 5 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 9
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Không có 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.
Issue khác của react-ui-org/react-ui
-
maintenance
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
react-ui-org/react-ui#784 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
accessibility bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
react-ui-org/react-ui#775 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Update ESLint to version 10Đang mởmaintenance
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 63/100
react-ui-org/react-ui#753 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Playwright container fails `devEngines` check again because its image does not pin Node.jsCó thể đã có người làm @bedrich-schindler đã nhận hôm nay. Đang mởbug maintenance
Độ khó 4/5 1-2 ngày Mức phù hợp với người mới 25/100
react-ui-org/react-ui#782 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Supported versionsĐang mởmaintenance
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
react-ui-org/react-ui#780 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của react-ui-org/react-ui
Issue tương tự
-
cloudflare-cache-binding: `unstorage://` keys are rejected by the Cache API, cache never storesĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
Maintainer thường phản hồi trong vòng 2 ngày
-
desktop: pasted image breaks and send fails with "Failed to fetch" after the clipboard temp file is deletedCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởreproduced
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
anomalyco/opencode#62052 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
strapi/design-system#2084 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày