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

Can't remove first dashboard (idDashboard=1) #8870

Closed
MagicFab opened this Issue Sep 29, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@MagicFab

MagicFab commented Sep 29, 2015

I can't remove the first Dashboard. If there is only one, "Remove Dashboard" and other options are not available.

If there are multiple Dashboards and I want to only remove the first one, confirmation is requested as usual, confirming does not delete the Dashboard.

@sgiehl

This comment has been minimized.

Show comment
Hide comment
@sgiehl

sgiehl Sep 30, 2015

Member

That one can't be removed as dashboard with id 1 will always be loaded by default. That way it is also sure that at least one dashboard exists for every user.
Guess we should disable the remove link for that one or display a message when the user tries to remove it.

Member

sgiehl commented Sep 30, 2015

That one can't be removed as dashboard with id 1 will always be loaded by default. That way it is also sure that at least one dashboard exists for every user.
Guess we should disable the remove link for that one or display a message when the user tries to remove it.

@mattab

This comment has been minimized.

Show comment
Hide comment
@mattab

mattab Oct 6, 2015

Member

Guess we should disable the remove link for that one or display a message when the user tries to remove it.

Or maybe we can "Grey out" the link, and on hover show a tooltip "The default dashboard cannot be removed." - what do you think?

Member

mattab commented Oct 6, 2015

Guess we should disable the remove link for that one or display a message when the user tries to remove it.

Or maybe we can "Grey out" the link, and on hover show a tooltip "The default dashboard cannot be removed." - what do you think?

@mattab mattab added this to the 2.15.1 milestone Oct 6, 2015

@mattab mattab added the c: Usability label Oct 6, 2015

@MagicFab

This comment has been minimized.

Show comment
Hide comment
@MagicFab

MagicFab Oct 15, 2015

@mattab +1 for your suggestion.

MagicFab commented Oct 15, 2015

@mattab +1 for your suggestion.

@sgiehl sgiehl self-assigned this Oct 15, 2015

sgiehl added a commit that referenced this issue Oct 17, 2015

@mattab mattab modified the milestones: 2.15.0, 2.15.1 Oct 19, 2015

@mattab mattab closed this in #9043 Oct 19, 2015

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