Page - update logic to apply sidebar-less layout
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
- 45/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- react, typescript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Xem xét logic layout không có sidebar trong component Page và hành vi được giới thiệu bởi pull request #11760. Theo dõi cách các sidebar props bị bỏ qua, falsey và được đặt rõ ràng thành null được xử lý, sau đó xác minh rằng modifier class được áp dụng cho các trường hợp dự kiến mà không làm hỏng các consumer hiện có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the enhancement or change
Followup to https://github.com/patternfly/patternfly-react/pull/11760 where we apply logic to apply a modifier class for sidebar-less layout if sidebar={null} (tldr reason being to avoid breaking consumers who may rely on sidebar being explicitly undefined, but want to keep the layout as-is).
We want to update the logic in the next breaking change so that omitting the sidebar prop (or it otherwise being falsey) will apply this modifier class instead of having to explicitly pass null to the prop.
Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?
Any other information?
- Ngôn ngữ chính
- TypeScript
- Star
- 862
- Fork
- 392
- Merge trung bình
- 4 ngày 8 giờ
- Pull request đã merge (30 ngày)
- 9
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 patternfly/patternfly-react
-
PF Team
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/patternfly-react#12651 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
patternfly/patternfly-react#12637 ·
-
PF Team
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
patternfly/patternfly-react#12617 · 1 bình luận ·
-
PF Team
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/patternfly-react#12610 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/patternfly-react#12600 ·
Tất cả issue của patternfly/patternfly-react
Issue tương tự
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Crush Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ElementsProject/cln-application#167 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Quantco/pnpm-licenses#17 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100