refactor: bed size is always defined

pedrolamas · pedrolamas · commit 907fdc18fed1 · 2025-02-26T14:10:17.000Z
Signed-off-by: Pedro Lamas &lt;pedrolamas@gmail.com&gt;
diff --git a/src/components/settings/timelapse/subsettings/CustomParkPositionSettings.vue b/src/components/settings/timelapse/subsettings/CustomParkPositionSettings.vue
@@ -108,14 +108,7 @@ export default class CustomParkPositionSettings extends Mixins(StateMixin) {
   }
 
   get bedSize (): BedSize {
-    const bedSize: BedSize | undefined = this.$store.getters['printer/getBedSize']
-
-    return bedSize ?? {
-      minX: -Infinity,
-      minY: -Infinity,
-      maxX: Infinity,
-      maxY: Infinity
-    }
+    return this.$store.getters['printer/getBedSize']
   }
 
   get settings (): TimelapseSettings {
diff --git a/src/components/widgets/bedmesh/BedMeshChart.vue b/src/components/widgets/bedmesh/BedMeshChart.vue
@@ -45,7 +45,7 @@ export default class BedMeshChart extends Mixins(BrowserMixin) {
     return this.$store.state.mesh.flatSurface
   }
 
-  get bedSize (): BedSize | undefined {
+  get bedSize (): BedSize {
     return this.$store.getters['printer/getBedSize']
   }
 
@@ -205,14 +205,14 @@ export default class BedMeshChart extends Mixins(BrowserMixin) {
       },
       xAxis3D: {
         type: 'value',
-        min: this.bedSize?.minX,
-        max: this.bedSize?.maxX,
+        min: this.bedSize.minX,
+        max: this.bedSize.maxX,
         ...axisCommon
       },
       yAxis3D: {
         type: 'value',
-        min: this.bedSize?.minY,
-        max: this.bedSize?.maxY,
+        min: this.bedSize.minY,
+        max: this.bedSize.maxY,
         ...axisCommon
       },
       zAxis3D: {
diff --git a/src/components/widgets/gcode-preview/GcodePreview.vue b/src/components/widgets/gcode-preview/GcodePreview.vue
@@ -560,14 +560,7 @@ export default class GcodePreview extends Mixins(StateMixin, BrowserMixin) {
   }
 
   get bedSize (): BedSize {
-    const bedSize: BedSize | undefined = this.$store.getters['printer/getBedSize']
-
-    return bedSize ?? {
-      minX: 0,
-      minY: 0,
-      maxX: 100,
-      maxY: 100
-    }
+    return this.$store.getters['printer/getBedSize']
   }
 
   get viewBox (): BBox {
diff --git a/src/components/widgets/toolhead/ToolheadControlCircle.vue b/src/components/widgets/toolhead/ToolheadControlCircle.vue
@@ -753,14 +753,7 @@ export default class ToolheadControlCircle extends Mixins(StateMixin, ToolheadMi
   }
 
   get bedSize (): BedSize {
-    const bedSize: BedSize | undefined = this.$store.getters['printer/getBedSize']
-
-    return bedSize ?? {
-      minX: 0,
-      minY: 0,
-      maxX: 0,
-      maxY: 0
-    }
+    return this.$store.getters['printer/getBedSize']
   }
 
   get bedCenter () {
diff --git a/src/store/printer/getters.ts b/src/store/printer/getters.ts
@@ -1020,16 +1020,9 @@ export const getters: GetterTree<PrinterState, RootState> = {
     ].includes(kinematics)
   },
 
-  getBedSize: (state): BedSize | undefined => {
+  getBedSize: (state): BedSize => {
     const { axis_minimum, axis_maximum } = state.printer.toolhead
 
-    if (
-      axis_minimum.length < 2 ||
-      axis_maximum.length < 2
-    ) {
-      return undefined
-    }
-
     const [minX, minY] = axis_minimum
     const [maxX, maxY] = axis_maximum
 

Original file line number	Diff line number	Diff line change
`@@ -108,14 +108,7 @@ export default class CustomParkPositionSettings extends Mixins(StateMixin) {`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`get bedSize (): BedSize {`
`111`		`- const bedSize: BedSize \| undefined = this.$store.getters['printer/getBedSize']`
`112`		`-`
`113`		`- return bedSize ?? {`
`114`		`- minX: -Infinity,`
`115`		`- minY: -Infinity,`
`116`		`- maxX: Infinity,`
`117`		`- maxY: Infinity`
`118`		`- }`
	`111`	`+ return this.$store.getters['printer/getBedSize']`
`119`	`112`	`}`
`120`	`113`
`121`	`114`	`get settings (): TimelapseSettings {`
Original file line number	Diff line number	Diff line change
`@@ -560,14 +560,7 @@ export default class GcodePreview extends Mixins(StateMixin, BrowserMixin) {`
`560`	`560`	`}`
`561`	`561`
`562`	`562`	`get bedSize (): BedSize {`
`563`		`- const bedSize: BedSize \| undefined = this.$store.getters['printer/getBedSize']`
`564`		`-`
`565`		`- return bedSize ?? {`
`566`		`- minX: 0,`
`567`		`- minY: 0,`
`568`		`- maxX: 100,`
`569`		`- maxY: 100`
`570`		`- }`
	`563`	`+ return this.$store.getters['printer/getBedSize']`
`571`	`564`	`}`
`572`	`565`
`573`	`566`	`get viewBox (): BBox {`
Original file line number	Diff line number	Diff line change
`@@ -753,14 +753,7 @@ export default class ToolheadControlCircle extends Mixins(StateMixin, ToolheadMi`
`753`	`753`	`}`
`754`	`754`
`755`	`755`	`get bedSize (): BedSize {`
`756`		`- const bedSize: BedSize \| undefined = this.$store.getters['printer/getBedSize']`
`757`		`-`
`758`		`- return bedSize ?? {`
`759`		`- minX: 0,`
`760`		`- minY: 0,`
`761`		`- maxX: 0,`
`762`		`- maxY: 0`
`763`		`- }`
	`756`	`+ return this.$store.getters['printer/getBedSize']`
`764`	`757`	`}`
`765`	`758`
`766`	`759`	`get bedCenter () {`