I don't think we'll prevent all users from mis-clicking the button with this change, but having a different Icon can probably avoid *some* confusion.
I don't really like that this icon would imply we 'pull the plug', but there is sadly not really a better icon that is different from the shutdown one. I'm a bit unsure myself, but *if* we want to have a different icon here than for the guest shutdown, consider this patch: Reviewed-by: Dominik Csapak <d.csa...@proxmox.com> On 7/2/25 11:52, Aaron Lauterer wrote:
this makes it optically different to the guest shutdown and hopefully helps to avoid confusion and accidential node shutdowns. Signed-off-by: Aaron Lauterer <a.laute...@proxmox.com> --- Notes: changes since: v1: * rebased with updated indentationsThis came up in the forum and I do like the idea and the change is smallenough to just quickly do it https://forum.proxmox.com/threads/disable-server-shutdown-restart-from-the-web-interface.12602/page-2#post-769235 www/manager6/node/Config.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/node/Config.js b/www/manager6/node/Config.js index 4cd1671c..8946a616 100644 --- a/www/manager6/node/Config.js +++ b/www/manager6/node/Config.js @@ -117,7 +117,7 @@ Ext.define('PVE.node.Config', { handler: function () { node_command('shutdown'); }, - iconCls: 'fa fa-power-off', + iconCls: 'fa fa-plug', });var shellBtn = Ext.create('PVE.button.ConsoleButton', {
_______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
