fix: opening formatted links

Author: paescuj

docs/CHANGELOG.md

@@ -16,6 +16,7 @@
 - `DX` - Tools submodules removed from the repository
 - `Improvement` - Shift + Down/Up will allow to select next/previous line instead of Inline Toolbar flipping
 - `Improvement` - The API `caret.setToBlock()` offset now works across the entire block content, not just the first or last node.
+- `Fix` - Opening links (via ctrl/cmd key + click) that are additionally formatted (e.g. bold)
 
 ### 2.30.7
 

src/components/dom.ts

@@ -556,16 +556,6 @@ export default class Dom {
     return element;
   }
 
-  /**
-   * Returns true if element is anchor (is A tag)
-   *
-   * @param {Element} element - element to check
-   * @returns {boolean}
-   */
-  public static isAnchor(element: Element): element is HTMLAnchorElement {
-    return element.tagName.toLowerCase() === 'a';
-  }
-
   /**
    * Return element's offset related to the document
    *

src/components/modules/ui.ts

@@ -767,23 +767,41 @@ export default class UI extends Module<UINodes> {
       return;
     }
 
-    /**
-     * case when user clicks on anchor element
-     * if it is clicked via ctrl key, then we open new window with url
-     */
-    const element = event.target as Element;
+    // Check if user clicks on a link while holding down the ctrl/cmd key.
+    // In that case, open it in a new tab/window.
     const ctrlKey = event.metaKey || event.ctrlKey;
 
-    if ($.isAnchor(element) && ctrlKey) {
-      event.stopImmediatePropagation();
-      event.stopPropagation();
+    if (ctrlKey && event.target instanceof Element) {
+      let currentElement: Element | null = event.target;
+      let anchor = null;
 
-      const href = element.getAttribute('href');
-      const validUrl = _.getValidUrl(href);
+      while (currentElement) {
+        if (currentElement === this.nodes.redactor) {
+          break;
+        }
 
-      _.openTab(validUrl);
+        if (currentElement.tagName === 'A') {
+          anchor = currentElement;
+          break;
+        }
 
-      return;
+        currentElement = currentElement.parentElement;
+      }
+
+      if (anchor) {
+        event.stopImmediatePropagation();
+        event.stopPropagation();
+
+        const href = anchor.getAttribute('href');
+
+        if (href !== null) {
+          const validUrl = _.getValidUrl(href);
+
+          window.open(validUrl, '_blank');
+        }
+
+        return;
+      }
     }
 
     this.processBottomZoneClick(event);

src/components/utils.ts

@@ -665,15 +665,6 @@ export function generateBlockId(): string {
   return nanoid(idLen);
 }
 
-/**
- * Opens new Tab with passed URL
- *
- * @param {string} url - URL address to redirect
- */
-export function openTab(url: string): void {
-  window.open(url, '_blank');
-}
-
 /**
  * Returns random generated identifier
  *