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

Shutdown btn functionality? #12

Closed
0verbyte opened this issue Jan 27, 2014 · 8 comments
Closed

Shutdown btn functionality? #12

0verbyte opened this issue Jan 27, 2014 · 8 comments

Comments

@0verbyte
Copy link
Contributor

I would suppose this should be able to shutdown the machine and reboot it? It would not make sense to just hard shutdown the machine without it rebooting.

Thoughts?

@gbb
Copy link

gbb commented Jan 27, 2014

I think it's an enormously bad idea to put a shutdown button on a monitoring page!

@gbb
Copy link

gbb commented Jan 27, 2014

I'd also recommend changing the 'power/shutdown button' into a 'refresh all' button at the top. That would be useful and in line with a monitoring dashboard.

@joseph-norman
Copy link

Agreed with what gbb said about it being fully functional! What was the original design idea for the power button?

@gbb
Copy link

gbb commented Jan 27, 2014

I suppose a key question is: is this a monitoring panel or a control panel?

@0verbyte
Copy link
Contributor Author

I will put this on hold until we decide further what the functionality should be. But here are some ideas for functionality:

  • display a modal dialog confirming you wish to restart the server
  • refresh all modules
  • nothing at all(remove the button)

@tariqbuilds
Copy link
Owner

The power button was naively intended to be a page refresh, nothing more. Adding a server shut-down or restart button to a monitoring panel seems out of place. Everything else is a consumption of data whereas shutdown/restart is an actionable option.

Personally speaking, I certainly would not want it on a monitoring panel for any of my servers.

However, I welcome icon alternatives.

@lacivert
Copy link
Contributor

@afaqurk very good idea for page refresh but I think 'refresh' button would be more understandable. Otherwise it looks confusing really. I was afraid to touch the button if my host go to restart state.

@tariqbuilds
Copy link
Owner

@lacivert - Fair enough. I have changed it to a refresh icon for now.

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

No branches or pull requests

5 participants