patternfly / patternfly/patternfly-react
Page - change notificationDrawer props to be more generic
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ả
re: https://github.com/patternfly/patternfly-react/issues/8541#issuecomment-1384624420
There are currently a props on the <Page> component (notificationDrawer and isNotificationDrawerExpanded) that attach a drawer to the page. AFAIK there is no requirement to use a notification drawer there - the props just update the page to support a drawer that appears over the main content area. Assuming we don't want to limit that use of a drawer in the page to notification drawer only, I wonder if it makes sense to rename those props to something more generic so it's clear it can be used for any sort of drawer content.
cc @mcarrano @mceledonia @tlabaj
Jira Issue: PF-141
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 từ component Page và tìm các props notificationDrawer và isNotificationDrawerExpanded cùng các nơi chúng được sử dụng. Xác nhận cách đặt tên chung được mong đợi cho nội dung drawer, cập nhật các tham chiếu bị ảnh hưởng và xác minh rằng hành vi drawer của Page vẫn không thay đổi.
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ái cấu trúc
- Độ 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
- 35/100