From 8b8970705278f37bec3ad91d905fd05a2c87d526 Mon Sep 17 00:00:00 2001
From: Zeanon <thezeanon@gmail.com>
Date: Sun, 22 Dec 2024 19:11:58 +0100
Subject: [PATCH] .

---
 package.json                                  |  2 +-
 .../Temperature/TemperaturePanelList.vue      | 13 +++++++
 src/components/ui/TopCornerMenuService.vue    | 35 ++++++++++++++++---
 3 files changed, 45 insertions(+), 5 deletions(-)

diff --git a/package.json b/package.json
index ff83d0e77..3d5eb7eb9 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "mainsail",
-    "version": "2.13.4",
+    "version": "2.13.3",
     "private": true,
     "decription": "a klipper web interface",
     "author": {
diff --git a/src/components/panels/Temperature/TemperaturePanelList.vue b/src/components/panels/Temperature/TemperaturePanelList.vue
index 8bab6117b..2aa3030a8 100644
--- a/src/components/panels/Temperature/TemperaturePanelList.vue
+++ b/src/components/panels/Temperature/TemperaturePanelList.vue
@@ -136,6 +136,8 @@ export default class TemperaturePanelList extends Mixins(BaseMixin) {
             // hide MCU & Host sensors, if the function is enabled
             if (this.hideMcuHostSensors && this.checkMcuHostSensor(fullName)) return false
 
+            if (this.mcuSensorDisconnected(fullName)) return false
+
             return !this.mpcBlockTemperatures.includes(fullName)
                 && !this.mpcAmbientTemperatures.includes(fullName)
         })
@@ -160,6 +162,17 @@ export default class TemperaturePanelList extends Mixins(BaseMixin) {
         return ['temperature_mcu', 'temperature_host'].includes(sensor_type)
     }
 
+    mcuSensorDisconnected(fullName: string) {
+        const settingsObject = this.settings[fullName.toLowerCase()] ?? {}
+        const sensor_type = settingsObject.sensor_type ?? ''
+
+        if (sensor_type != "temperature_mcu") return false
+        if (!(("mcu " + settingsObject.sensor_mcu) in this.$store.state.printer)) return false
+
+        const mcu = this.$store.state.printer["mcu " + settingsObject.sensor_mcu]
+        return mcu.non_critical_disconnected ?? false
+    }
+
     filterNamesAndSort(fullNames: string[]) {
         return fullNames.filter(this.isVisibleName).sort(this.sortObjectName)
     }
diff --git a/src/components/ui/TopCornerMenuService.vue b/src/components/ui/TopCornerMenuService.vue
index e87dcc78a..9af5eb81a 100644
--- a/src/components/ui/TopCornerMenuService.vue
+++ b/src/components/ui/TopCornerMenuService.vue
@@ -19,6 +19,14 @@
                 <v-icon small>{{ mdiStop }}</v-icon>
             </v-btn>
         </v-list-item-action>
+        <confirmation-dialog
+            :show="showStartDialog"
+            :title="dialogStartTitle"
+            :text="dialogStartDescription"
+            :action-button-text="$t('App.TopCornerMenu.Start')"
+            :cancel-button-text="$t('App.TopCornerMenu.Cancel')"
+            @action="serviceStart"
+            @close="showStartDialog = false" />
         <confirmation-dialog
             :show="showRestartDialog"
             :title="dialogRestartTitle"
@@ -55,6 +63,7 @@ export default class TopCornerMenuService extends Mixins(BaseMixin, ServiceMixin
 
     showRestartDialog = false
     showStopDialog = false
+    showStartDialog = false
 
     get name() {
         if (this.hideOtherInstances && this.service === this.klipperInstance) return 'Klipper'
@@ -79,6 +88,10 @@ export default class TopCornerMenuService extends Mixins(BaseMixin, ServiceMixin
         return null
     }
 
+    get dialogStartTitle() {
+        return this.$t('App.TopCornerMenu.ConfirmationDialog.Title.ServiceStart')
+    }
+
     get dialogRestartTitle() {
         if (this.service === this.klipperInstance)
             return this.$t('App.TopCornerMenu.ConfirmationDialog.Title.KlipperRestart')
@@ -90,18 +103,22 @@ export default class TopCornerMenuService extends Mixins(BaseMixin, ServiceMixin
         return this.$t('App.TopCornerMenu.ConfirmationDialog.Title.ServiceStop')
     }
 
+    get dialogStartDescription() {
+        return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.ServiceStart', {serviceName: this.service}).toString()
+    }
+
     get dialogRestartDescription() {
         if (this.service === this.klipperInstance)
             return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.KlipperRestart')
 
-        return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.ServiceRestart')
+        return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.ServiceRestart', {serviceName: this.service}).toString()
     }
 
     get dialogStopDescription() {
         if (this.service === this.klipperInstance)
             return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.KlipperStop')
 
-        return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.ServiceStop')
+        return this.$t('App.TopCornerMenu.ConfirmationDialog.Description.ServiceStop', {serviceName: this.service}).toString()
     }
 
     get disableStopButton() {
@@ -113,8 +130,12 @@ export default class TopCornerMenuService extends Mixins(BaseMixin, ServiceMixin
     }
 
     clickStart() {
-        this.$socket.emit('machine.services.start', { service: this.service })
-        this.closeMenu()
+        if (this.printerIsPrinting) {
+            this.showStartDialog = true
+            return
+        }
+
+        this.serviceStart()
     }
 
     clickRestart() {
@@ -135,6 +156,12 @@ export default class TopCornerMenuService extends Mixins(BaseMixin, ServiceMixin
         this.serviceStop()
     }
 
+    serviceStart() {
+        this.showStartDialog = false
+        this.$socket.emit('machine.services.start', { service: this.service })
+        this.closeMenu()
+    }
+
     serviceRestart() {
         this.showRestartDialog = false
         this.$socket.emit('machine.services.restart', { service: this.service })