Hopefully final version

Alto/Classes/Models/AltoWebView/Scripts/src/ContextMenuHandler.js

@@ -1,105 +1,113 @@
-function findAllMediaAndLinks(el) {
-  const found = {
-	link: null,
-	image: null,
-	video: null,
-	text: null
-  };
-
-  while (el) {
-	if (el.nodeType === Node.TEXT_NODE) {
-	  el = el.parentElement;
-	  continue;
-	}
-
-	if (!found.link && el.tagName === "A") {
-	  found.link = el;
-	}
-
-	if (!found.image && el.tagName === "IMG") {
-	  found.image = el;
-	}
-
-	if (!found.video && el.tagName === "VIDEO") {
-	  found.video = el;
-	}
-
-	if (!found.text && el.textContent && el.textContent.trim().length > 0) {
-	  found.text = el;
-	}
+(function() {
+  function findAllMediaAndLinks(el) {
+	var found = {
+	  link: null,
+	  image: null,
+	  video: null,
+	  text: null
+	};
 
-	if (found.link && found.image && found.video) break;
+	while (el) {
+	  if (el.nodeType === Node.TEXT_NODE) {
+		el = el.parentElement;
+		continue;
+	  }
 
-	el = el.parentElement;
-  }
+	  if (!found.link && el.tagName === "A") {
+		found.link = el;
+	  }
 
-  return found;
-}
+	  if (!found.image && el.tagName === "IMG") {
+		found.image = el;
+	  }
 
-window.oncontextmenu = function(event) {
-  event.preventDefault();
+	  if (!found.video && el.tagName === "VIDEO") {
+		found.video = el;
+	  }
 
-  const found = findAllMediaAndLinks(event.target);
-  const selection = window.getSelection();
-  const selectedText = selection.toString().trim();
+	  if (
+		!found.text &&
+		el.children.length === 0 &&
+		el.textContent &&
+		el.textContent.trim().length > 0
+	  ) {
+		found.text = el;
+	  }
 
-  let message = null;
+	  if (found.link && found.image && found.video) break;
 
-  const isSelectionInside =
-	selectedText.length > 0 &&
-	found.text &&
-	selection.anchorNode &&
-	found.text.contains(selection.anchorNode);
+	  el = el.parentElement;
+	}
 
-  if (isSelectionInside) {
+	return found;
+  }
 
-	if (found.link) {
+  window.addEventListener("contextmenu", function(event) {
+	event.preventDefault();
+
+	var found = findAllMediaAndLinks(event.target);
+	var selection = window.getSelection();
+	var selectedText = selection ? selection.toString().trim() : "";
+
+	var message = null;
+
+	var isSelectionInside =
+	  selectedText.length > 0 &&
+	  found.text &&
+	  selection &&
+	  selection.anchorNode &&
+	  found.text.contains(selection.anchorNode);
+
+	if (isSelectionInside) {
+	  if (found.link) {
+		message = {
+		  type: "link",
+		  value: found.link.href,
+		  containsImage: !!found.image
+		};
+	  } else {
+		message = {
+		  type: "text",
+		  value: selectedText
+		};
+	  }
+	} else if (found.link) {
 	  message = {
 		type: "link",
 		value: found.link.href,
 		containsImage: !!found.image
 	  };
-	} else {
+	} else if (found.image) {
+	  message = {
+		type: "image",
+		value: found.image.src
+	  };
+	} else if (found.video) {
+	  var videoSrc = found.video.src || (found.video.querySelector('source') && found.video.querySelector('source').src) || null;
+	  message = {
+		type: "video",
+		value: videoSrc || "no-src"
+	  };
+	} else if (found.text) {
 	  message = {
 		type: "text",
-		value: selectedText,
+		value: found.text.textContent.trim()
+	  };
+	} else {
+	  message = {
+		type: "unknown",
+		value: ""
 	  };
 	}
-  } else if (found.link) {
-	message = {
-	  type: "link",
-	  value: found.link.href,
-	  containsImage: !!found.image
-	};
-  } else if (found.image) {
-	message = {
-	  type: "image",
-	  value: found.image.src,
-	};
-  } else if (found.video) {
-	const videoSrc = found.video.src || (found.video.querySelector('source')?.src) || null;
-	message = {
-	  type: "video",
-	  value: videoSrc || "no-src",
-	};
-  } else if (found.text) {
-	message = {
-	  type: "text",
-	  value: found.text.textContent.trim(),
-	};
-  } else {
-	message = {
-	  type: "unknown",
-	  value: "",
-	};
-  }
 
-  if (window.webkit?.messageHandlers?.menuHandler) {
-	window.webkit.messageHandlers.menuHandler.postMessage(message);
-  } else {
-	console.log("Handler not found", message);
-  }
 
-  return false;
-};
+	if (window.webkit?.messageHandlers?.menuHandler) {
+		window.webkit.messageHandlers.menuHandler.postMessage(message);
+	  } else {
+		console.log("Handler not found", message);
+	}
+
+	return false;
+  });
+})();