patternfly / patternfly/patternfly-react
Drawer - Refine Drawer Usage for Better Integration with External Components
Chưa có ai nhận issue này.
- 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 problem
This arose from fixing a layout bug with the drawer component in PR #4142.
Using DrawerPanelContent within an externally provided DrawerPanelContent (in this case from PF Topology) breaks the layout of the sidebar panel.
- Addressed by removing the nested
DrawerPanelContentand directly applyingDrawerHeadandDrawerPanelBodywithin theDrawerPanelContentthat is externally provided from TopologyView.
In our use case, the utility class pf-v6-u-p-md was applied to get the same padding, but this workaround highlights a gap in the current implementation.
How do you reproduce the problem?
See https://github.com/opendatahub-io/odh-dashboard/pull/4142.
Expected behavior
PatternFly needs an improved way to structure and compose standard drawer elements like DrawerHead and DrawerPanelBody within an external DrawerPanelContent wrapper from the TopologyView.
Is this issue blocking you? No.
What is your product and what release date are you targeting?
RHOAI
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
Bắt đầu với PR #4142 và cách TopologyView sử dụng DrawerPanelContent được cung cấp từ bên ngoài. Đọc cách DrawerHead và DrawerPanelBody được kết hợp bên trong các thành phần drawer, sau đó tái hiện kịch bản bố cục sidebar. Hoàn tất khi các phần tử drawer tiêu chuẩn tích hợp với wrapper bên ngoài mà không cần workaround nested-content hoặc class padding thủ công.
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ó
- 4/5
- Thời gian dự kiến
- 3-5 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
- 35/100