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

Jump to bottom

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

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

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

MagicFab opened this issue Sep 29, 2015 · 3 comments
Assignees
@sgiehl
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently.
Milestone
2.15.0

Comments

@MagicFab
Copy link

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
Copy link
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
Copy link
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 For issues that let users achieve a defined goal more effectively or efficiently. label Oct 6, 2015
@MagicFab
Copy link
Author

@mattab +1 for your suggestion.

@sgiehl sgiehl self-assigned this Oct 15, 2015
sgiehl added a commit that referenced this issue Oct 17, 2015
@sgiehl
fixes #8870 improved remove dashboard behaviour for default dashboard
37a3ca9
@sgiehl sgiehl mentioned this issue Oct 17, 2015
Merged
@mattab mattab modified the milestones: 2.15.0, 2.15.1 Oct 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgiehl sgiehl

Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently.
Projects
None yet
Milestone
2.15.0
Development

No branches or pull requests
3 participants
@MagicFab @mattab @sgiehl