Merge pull request microsoft#41242 from mlewand/open-link-multi-selections

jrieken · web-flow · commit 5f3a2e87a333 · 2018-01-09T00:08:50.000+01:00
Added support for multiple selections in Open Link action
diff --git a/src/vs/editor/contrib/links/links.ts b/src/vs/editor/contrib/links/links.ts
@@ -396,9 +396,14 @@ class OpenLinkAction extends EditorAction {
 			return;
 		}
 
-		let link = linkDetector.getLinkOccurrence(editor.getPosition());
-		if (link) {
-			linkDetector.openLinkOccurrence(link, false);
+		let selections = editor.getSelections();
+
+		for (let sel of selections) {
+			let link = linkDetector.getLinkOccurrence(sel.getEndPosition());
+
+			if (link) {
+				linkDetector.openLinkOccurrence(link, false);
+			}
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -396,9 +396,14 @@ class OpenLinkAction extends EditorAction {`
`396`	`396`	`return;`
`397`	`397`	`}`
`398`	`398`
`399`		`- let link = linkDetector.getLinkOccurrence(editor.getPosition());`
`400`		`- if (link) {`
`401`		`- linkDetector.openLinkOccurrence(link, false);`
	`399`	`+ let selections = editor.getSelections();`
	`400`	`+`
	`401`	`+ for (let sel of selections) {`
	`402`	`+ let link = linkDetector.getLinkOccurrence(sel.getEndPosition());`
	`403`	`+`
	`404`	`+ if (link) {`
	`405`	`+ linkDetector.openLinkOccurrence(link, false);`
	`406`	`+ }`
`402`	`407`	`}`
`403`	`408`	`}`
`404`	`409`	`}`