small fixes (code review)

src/packages/printing/Printing.tsx

@@ -17,7 +17,7 @@ import { PackageIntl } from "@open-pioneer/runtime";
 import { useIntl, useService } from "open-pioneer:react-hooks";
 import { FC, useEffect, useState } from "react";
 import { FileFormatType, PrintingController } from "./PrintingController";
-import { ViewPaddingBehavior, type PrintingService } from "./index";
+import type { ViewPaddingBehavior, PrintingService } from "./index";
 
 const LOG = createLogger("printing");
 
@@ -31,7 +31,7 @@ export interface PrintingProps extends CommonComponentProps {
     mapId: string;
 
     /**
-     * Whether to respect the map's padding when printing (default: `"auto"`).
+     * Whether to respect the map view's padding when printing (default: `"auto"`).
      *
      * See also {@link ViewPaddingBehavior}.
      */

src/packages/printing/PrintingServiceImpl.ts

@@ -125,8 +125,8 @@ export class PrintJob {
         }
 
         // Position the scale bar manually.
-        // The 50px should be plenty to avoid overlapping with openlayers attributions on most cases.
-        // Additionally take the view padding into account (if behavior is 'auto').
+        // The 50px should be plenty to avoid overlapping with open layers attributions on most cases.
+        // Additionally, take the view padding into account (if behavior is 'auto').
         let bottom = 50;
         let left = 8;
         if (this.viewPadding === "auto") {
@@ -220,9 +220,8 @@ export class PrintJob {
         newCanvas.width = width - padding.left - padding.right;
         newCanvas.height = height - padding.top - padding.bottom;
 
-        const ctx = canvas.getContext("2d");
         const newCtx = newCanvas.getContext("2d");
-        if (!ctx || !newCtx) {
+        if (!newCtx) {
             throw new Error("Failed to get a canvas context");
         }
 

src/packages/printing/index.ts

@@ -22,13 +22,13 @@ export interface PrintingOptions {
     overlayText?: string;
 
     /**
-     * Whether to respect the map's padding when printing (default: `"auto"`).
+     * Whether to respect the map view's padding when printing (default: `"auto"`).
      */
     viewPadding?: ViewPaddingBehavior;
 }
 
 /**
- * Whether to respect the map's padding when printing.
+ * Whether to respect the map view's padding when printing.
  *
  * - `"auto"`: Respect the map's current viewPadding.
  *   Padded regions of the map will _not_ be included in the print result.