patternfly / patternfly/patternfly-react
Bug - AlertGroup - li elements never unmount for timedout alerts
还没有人认领这个 Issue。
- 主要语言
- TypeScript
- 星标
- 862
- 派生
- 392
- 平均合并
- 4 天 8 小时
- 30 天内合并 PR
- 9
描述
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?
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 AlertGroup 组件开始,使用 alert timeout 示例重现该问题。在超时的 Alerts 消失时检查 DOM,并追踪它们的包装 li 元素是如何渲染和移除的。当超时的 Alerts 不再留下空的 li 元素时即表示完成,包括添加另一个 Alert 的情况。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- react, typescript
- 领域
- frontend
- Issue 类型
- 缺陷
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 停滞
- 描述清晰度
- 描述清楚
- 新手友好度
- 45/100