patternfly / patternfly/patternfly-react

Bug - AlertGroup - li elements never unmount for timedout alerts

Open
#11,496 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

PF Team Stale
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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.