patternfly / patternfly/patternfly-react
Bug - AlertGroup - li elements never unmount for timedout alerts
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
When Alerts with a timeout are rendered inside an AlertGroup, the wrapper li element for the Alerts never unmounts when the Alerts do. This causes empty li elements to remain in the DOM.
How do you reproduce the problem?
Go to the alert timeout example and add an alert in the example. Inspect the DOM and notice that when the Alert itself unmounts, the wrapper li element remains with nothing inside of it. Add another Alert and notice that the empty li still remains in addition to a new li with the new Alert rendered inside of it.
Expected behavior
The li elements would unmount the same as the Alerts themselves
Is this issue blocking you?
List the workaround if there is one.
Screenshots
If applicable, add screenshots to help explain the issue.
What is your environment?
- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
What is your product and what release date are you targeting?
Any other information?
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 component AlertGroup và tái hiện vấn đề bằng ví dụ về thời gian chờ của alert. Kiểm tra DOM khi các Alerts hết thời gian chờ biến mất và theo dõi cách các phần tử li wrapper của chúng được render và xóa. Hoàn thành khi các Alerts hết thời gian chờ không còn để lại các phần tử li trống, kể cả khi một Alert khác được thêm vào.
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
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 45/100