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

remove widget in dashboard works only once #3202

Closed
araker opened this issue Dec 19, 2018 · 2 comments · Fixed by #3722
Closed

remove widget in dashboard works only once #3202

araker opened this issue Dec 19, 2018 · 2 comments · Fixed by #3722
Assignees
Milestone

Comments

@araker
Copy link

araker commented Dec 19, 2018

Issue Summary

When removing multiple widgets in a dashboard, I can only perform the action once. The second time I click on the 'cross' of a widget the 'are you sure' dialog doesn't appear and the widget isn't removed. When I refresh the page, I can remove one widget again.

Steps to Reproduce

  1. Create a dashboard
  2. add two widgets
  3. remove the first widget
  4. remove the second widget, this doesn't work

I expect that I can remove multiple widgets, without refreshing the page in between.

Technical details:

  • Redash Version: 6.0.0+b8537
  • Browser/OS: chromium 66.3359.139 Ubuntu
  • How did you install Redash: Ami docker version
@arikfr arikfr added this to the v7.0.0 milestone Dec 19, 2018
@arikfr arikfr modified the milestones: v7.0.0, Next Mar 5, 2019
@arikfr
Copy link
Member

arikfr commented Mar 5, 2019

@ranbena no point in addressing this now, but just make sure to address it when working on the React grid.

@kravets-levko
Copy link
Collaborator

kravets-levko commented Mar 5, 2019

@arikfr @ranbena This issue is related to gridster library which we use to dashboard layout - sometimes it hooks up a bit more events than needed, and our event handlers stop working. I tried to find a fix for it some time ago, but after migrating to React this problem should just disappear (if React component will not have the same bug 🙂)

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

Successfully merging a pull request may close this issue.

4 participants