Merge pull request #29 from utomic-media/fix/unified-link-targets

Fix/unified link targets
utomic-media · Mar 24, 2023 · c6bc826 · c6bc826
2 parents bf2f4ca + 8e1647d
commit c6bc826
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/display/display.vue b/src/display/display.vue
@@ -6,6 +6,8 @@
 			class="dynamic-wrapper"
 			:href="computedLink"
 			v-tooltip.left="actionTooltip"
+			target="_blank"
+			rel="noopener noreferrer"
 		>
 			<span 
 				:class="hasValueClickAction ? 'action-background' : ''"
@@ -115,6 +117,11 @@ function valueClickAction(e: Event) {
 		e.stopPropagation();
 		copyValue();
 	} 
+
+	if (props.clickAction === 'link') {
+		// We opened a link in a new tab and don't want to get into the details view of the item
+		e.stopPropagation();
+	}
 	// else go on with the default events
 }
 

diff --git a/src/interface/interface.vue b/src/interface/interface.vue
@@ -166,6 +166,12 @@ function valueClickAction(e: Event) {
 		e.stopPropagation();
 		copyValue();
 	} 
+
+	if (props.clickAction === 'link' && props.disabled && props.value) {
+		e.stopPropagation();
+		window.open(computedLink.value, '_blank', 'noopener, noreferrer');
+	}
+	
 	// else go on with the default events
 }