fix: you should not update or restart services during a print

Author: cadriel

src/components/VersionStatus.vue

@@ -33,7 +33,7 @@
       <span>indicates a detached head, not on master or an invalid origin</span>
     </v-tooltip>
 
-    <v-btn v-if="hasUpdate && (!dirty && valid)" :disabled="disabled" :loading="loading" x-small text color="warning" @click="$emit('on-update')">UPDATE</v-btn>
+    <v-btn v-if="hasUpdate && !dirty && valid" :disabled="disabled" :loading="loading" x-small text color="warning" @click="$emit('on-update')">UPDATE</v-btn>
   </div>
 </template>
 

src/components/widgets/SystemCommandsWidget.vue

@@ -8,18 +8,25 @@
           <v-list-item-title>Host</v-list-item-title>
         </v-list-item-content>
       </template>
-      <v-list-item @click="confirmRebootDialog.open = true">
+
+      <v-list-item
+        @click="confirmRebootDialog.open = true"
+        :disabled="printerPrinting">
         <v-list-item-title>Reboot</v-list-item-title>
         <v-list-item-icon>
-          <v-icon>$powerCycle</v-icon>
+          <v-icon color="error">$powerCycle</v-icon>
         </v-list-item-icon>
       </v-list-item>
-      <v-list-item @click="confirmShutdownDialog.open = true">
+
+      <v-list-item
+        @click="confirmShutdownDialog.open = true"
+        :disabled="printerPrinting">
         <v-list-item-title>Shutdown</v-list-item-title>
         <v-list-item-icon>
-          <v-icon>$power</v-icon>
+          <v-icon color="error">$power</v-icon>
         </v-list-item-icon>
       </v-list-item>
+
     </v-list-group>
     <v-list-group
       prepend-icon="$restart"
@@ -32,21 +39,28 @@
       <v-list-item @click="serviceRestartMoonraker">
         <v-list-item-title>Restart Moonraker</v-list-item-title>
         <v-list-item-icon>
-          <v-icon>$restart</v-icon>
+          <v-icon color="warning">$restart</v-icon>
         </v-list-item-icon>
       </v-list-item>
-      <v-list-item @click="serviceRestartKlipper">
+
+      <v-list-item
+        @click="serviceRestartKlipper"
+        :disabled="printerPrinting">
         <v-list-item-title>Restart Klipper</v-list-item-title>
         <v-list-item-icon>
-          <v-icon>$restart</v-icon>
+          <v-icon color="error">$restartAlert</v-icon>
         </v-list-item-icon>
       </v-list-item>
-      <v-list-item @click="serviceFirmwareRestartKlippy">
+
+      <v-list-item
+        @click="serviceFirmwareRestartKlippy"
+        :disabled="printerPrinting">
         <v-list-item-title>Restart MCU(s)</v-list-item-title>
         <v-list-item-icon>
-          <v-icon>$restartAlert</v-icon>
+          <v-icon color="error">$restartAlert</v-icon>
         </v-list-item-icon>
       </v-list-item>
+
       <!-- <v-list-item @click="serverRestart">
         <v-list-item-title>Server Restart</v-list-item-title>
         <v-list-item-icon>

src/components/widgets/SystemVersionsWidget.vue

@@ -31,7 +31,7 @@
           <v-list-item-action-text>
             <version-status
               :has-update="hasUpdate(i)"
-              :disabled="hasWait(waits.onUpdate)"
+              :disabled="hasWait(waits.onUpdate) || printerPrinting"
               :loading="hasWait(waits.onUpdate)"
               :dirty="(component.name) ? false : component.is_dirty"
               :valid="(component.name) ? true : component.is_valid"
@@ -44,7 +44,7 @@
 
     <v-list-item class="v-list-item--x-dense">
       <v-list-item-content>
-        <v-btn text outlined x-small @click="updateComponent('system')">update os packages</v-btn>
+        <v-btn text outlined x-small :disabled="printerPrinting" @click="updateComponent('system')">update os packages</v-btn>
       </v-list-item-content>
     </v-list-item>