patternfly / patternfly/patternfly-react
Bug - AlertGroup - li elements never unmount for timedout alerts
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 862
- Forks
- 392
- Avg merge
- 4d 8h
- Merged PRs (30d)
- 9
Description
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?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the AlertGroup component and reproduce the issue using the alert timeout example. Inspect the DOM as timed-out Alerts disappear and trace how their wrapper li elements are rendered and removed. Done means timed-out Alerts no longer leave empty li elements, including when another Alert is added.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100