patternfly / patternfly/patternfly-react

Page - update logic to apply sidebar-less layout

Đang mở
#11,797 0 bình luận 1 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Breaking change :boom: PF Team
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

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?

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

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.

Đánh giá

Công nghệ
react, typescript
Lĩnh vực
frontend
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.