Log in collapsible sections cause content to not render
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ 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
Start with the LogViewer and ExpandableSection interaction, then reproduce the failure using the linked CodeSandbox, especially with a ref and with the section initially hidden. Investigate why content does not render on the first hidden render; done means LogViewer content renders correctly when the section is opened, with and without a ref.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
When using the LogViewer within an ExpandableSection component it might not render correctly. Assigning a ref to the LogViewer makes it happen even more often.
Here is a sandbox setup with a ref that showcases that issue
https://codesandbox.io/s/focused-hooks-m95046?file=/index.tsx
Here is a video of it happening for me in Storybook using ref
Here is a picture of it happening with two nested Cards and one ExpandableSection, without ref
Comment from Juntao Wang from PatternFly Slack
Hi, this is definitely a bug in the react-log-viewer, could you please create an issue in https://github.com/patternfly/react-log-viewer/issues? it’s a little bit tricky because the styles are setting everywhere and seems there are some render problems when using the log viewer with the expandable sections. it will not render correctly when the section is hidden for the first time.
The current workaround is creating a custom component that wraps the LogViewer and setting the ref there, and use React.memo on the whole component. Finally, set that component as the child in the ExpandableSection . However, in this situation, you may need to set the initial state of the isExpanded to true to make the initial render correctly.
I will be on PTO starting tomorrow for ~10 days so I might not have time to do a deeper investigation into this issue but I will check it again after I am back to see if I could find anything. Could you also take a look on this issue when you are available
Jira Issue: PF-2011
- Ngôn ngữ chính
- TypeScript
- Star
- 34
- Fork
- 19
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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/react-log-viewer
-
Search within Log viewer can cause the parent container to shift and break the layout, when embedded in a scrollable parentCó thể đã có người làm @fao89 đã nhận 136 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 67/100
patternfly/react-log-viewer#106 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
patternfly/react-log-viewer#101 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
patternfly/react-log-viewer#98 · 1 bình luận ·
-
How to set search text by props?Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
patternfly/react-log-viewer#95 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
patternfly/react-log-viewer#91 · 1 bình luận ·
Tất cả issue của patternfly/react-log-viewer
Issue tương tự
-
submodule-pointer-regression
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
smith-horn/skillsmith#3061 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: ops type: test
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
accensa/x402-facilitator-stellar#559 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Fix the no-pin ruling in line-drawings: pin a below-the-record stage at the record's chapterĐang mởdocumentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
cosimochellini/one-piece-zero-spoiler#551 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
getWatched() omits __proto__ directories when cwd is setCó thể đã có người làm @maxazure đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
-
area:web enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày