src/display/editor/tools.js

@@ -1680,7 +1680,6 @@ class AnnotationEditorUIManager {
     }
 
     this.#updateModeCapability = Promise.withResolvers();
-    this.#currentDrawingSession?.commitOrRemove();
 
     this.#mode = mode;
     if (mode === AnnotationEditorType.NONE) {

src/display/touch_manager.js

@@ -215,12 +215,9 @@ class TouchManager {
     if (evt.touches.length >= 2) {
       return;
     }
-    // #touchMoveAC shouldn't be null but it seems that irl it can (see #19793).
-    if (this.#touchMoveAC) {
-      this.#touchMoveAC.abort();
-      this.#touchMoveAC = null;
-      this.#onPinchEnd?.();
-    }
+    this.#touchMoveAC.abort();
+    this.#touchMoveAC = null;
+    this.#onPinchEnd?.();
 
     if (!this.#touchInfo) {
       return;

web/signature_manager.css

@@ -163,7 +163,6 @@
   --thickness-bg: var(--dialog-bg-color);
   --thickness-label-color: var(--primary-color);
   --thickness-slider-color: var(--primary-color);
-  --thickness-border: none;
   --draw-cursor: url(images/cursor-editorInk.svg) 0 16, pointer;
 
   @media (prefers-color-scheme: dark) {
@@ -220,7 +219,6 @@
     --thickness-bg: Canvas;
     --thickness-label-color: CanvasText;
     --thickness-slider-color: ButtonText;
-    --thickness-border: 1px solid var(--border-color);
   }
 
   #addSignatureDialogLabel {
@@ -412,18 +410,12 @@
               align-items: center;
               justify-content: center;
               gap: 8px;
-              padding: 6px 8px 7px;
+              padding: 6px 8px;
               margin: 0;
               background-color: var(--thickness-bg);
               border-radius: 4px 4px 0 0;
-              border-inline: var(--thickness-border);
-              border-top: var(--thickness-border);
               pointer-events: auto;
 
-              /* Move the div in order to cover to bottom border of the area. */
-              position: relative;
-              top: 1px;
-
               > label {
                 color: var(--thickness-label-color);
               }