Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(alerts): after dismissing alert are not removing from DOM #3608

Closed
YevheniiaMazur opened this issue Jan 31, 2018 · 2 comments
Closed

fix(alerts): after dismissing alert are not removing from DOM #3608

YevheniiaMazur opened this issue Jan 31, 2018 · 2 comments

Comments

@YevheniiaMazur
Copy link

Bug description or feature request:

WTR

  • Go to Alerts demo page
  • Scroll down to any alert with dismissible option (Dismissing demo or Dismiss on timeout)
  • Find alert in DOM
  • Dismiss alert (or wait time, after what it dismissed)
  • Look at DOM again
  • Add new alert
  • Repeat steps 3-6 several times
    alertnotremoved
    multiplealertsatdom

Expected result

All alerts after dismissing are removed from DOM

Versions of ngx-bootstrap, Angular, and Bootstrap:

ngx-bootstrap: all supported

Angular: all supported

Bootstrap: all supported

IraErshova added a commit to IraErshova/ngx-bootstrap that referenced this issue Feb 1, 2018
IraErshova added a commit to IraErshova/ngx-bootstrap that referenced this issue Feb 5, 2018
IraErshova added a commit to IraErshova/ngx-bootstrap that referenced this issue Feb 8, 2018
IraErshova added a commit to IraErshova/ngx-bootstrap that referenced this issue Feb 8, 2018
IraErshova added a commit to IraErshova/ngx-bootstrap that referenced this issue Feb 22, 2018
@ghost ghost removed the in progress label Feb 22, 2018
valorkin pushed a commit that referenced this issue Feb 22, 2018
…3622)

Fix removing alert from DOM after its dismissing

Close #3608
@martibenj
Copy link

Is there some update about this issue ? Will it be released ?

@valorkin
Copy link
Member

I suppose it's already published

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants