fix(link-tool): call clear function to reset link-tool state

Author: akulistus

src/components/inline-tools/inline-tool-link.ts

@@ -172,21 +172,11 @@ export default class LinkInlineTool implements InlineTool {
        * Unlink icon pressed
        */
       if (parentAnchor) {
-        /**
-         * If input is not opened, treat click as explicit unlink action.
-         * If input is opened (e.g., programmatic close when switching tools), avoid unlinking.
-         */
-        if (!this.inputOpened) {
-          this.selection.expandToTag(parentAnchor);
-          this.unlink();
-          this.closeActions();
-          this.checkState();
-          this.toolbar.close();
-        } else {
-          /** Only close actions without clearing saved selection to preserve user state */
-          this.closeActions(false);
-          this.checkState();
-        }
+        this.selection.expandToTag(parentAnchor);
+        this.unlink();
+        this.closeActions();
+        this.checkState();
+        this.toolbar.close();
 
         return;
       }
@@ -270,14 +260,16 @@ export default class LinkInlineTool implements InlineTool {
     if (this.selection.isFakeBackgroundEnabled) {
       // if actions is broken by other selection We need to save new selection
       const currentSelection = new SelectionUtils();
-
       currentSelection.save();
 
       this.selection.restore();
       this.selection.removeFakeBackground();
 
-      // and recover new selection after removing fake background
-      currentSelection.restore();
+      // check if other selection happend
+      if (!currentSelection.savedSelectionRange.collapsed) {
+        // and recover new selection after removing fake background
+        currentSelection.restore();
+      }
     }
 
     this.nodes.input.classList.remove(this.CSS.inputShowed);

src/components/modules/toolbar/inline.ts

@@ -391,6 +391,9 @@ export default class InlineToolbar extends Module<InlineToolbarNodes> {
           onActivate: () => {
             this.toolClicked(instance);
           },
+          onClear: () => {
+            instance?.clear?.();
+          },
           hint: {
             title: toolTitle,
             description: shortcutBeautified,

src/components/utils/popover/components/popover-item/popover-item.ts

@@ -25,6 +25,21 @@ export abstract class PopoverItem {
     }
   }
 
+  /**
+   * Calls instance clear function
+   */
+  public clear(): void {
+    if (this.params === undefined) {
+      return;
+    }
+
+    if (!('onClear' in this.params)) {
+      return;
+    }
+
+    this.params.onClear?.(this.params);
+  }
+
   /**
    * Destroys the instance
    */

src/components/utils/popover/popover-inline.ts

@@ -167,9 +167,9 @@ export class PopoverInline extends PopoverDesktop {
     if (item !== this.nestedPopoverTriggerItem) {
       /**
        * In case tool had special handling for toggling button (like link tool which modifies selection)
-       * we need to call handleClick on nested popover trigger item
+       * we need to call clear on nested popover trigger item to restore initial state
        */
-      this.nestedPopoverTriggerItem?.handleClick();
+      this.nestedPopoverTriggerItem?.clear();
 
       /**
        * Then close the nested popover

types/utils/popover/popover-item.d.ts

@@ -178,6 +178,13 @@ export interface PopoverItemDefaultBaseParams {
    * @param event - event that initiated item activation
    */
   onActivate: (item: PopoverItemParams, event?: PointerEvent) => void;
+
+  /**
+   * Popover item clear handler
+   * 
+   * @param item - item to be cleared
+   */
+  onClear: (item: PopoverItemParams) => void;
 }
 
 /**