Fix wrapping overflow when up against the edge

Tyriar · Tyriar · commit c80b8a3318e6 · 2020-08-12T16:18:37.000-07:00
Fixes microsoft#104508
diff --git a/src/vs/workbench/services/hover/browser/hoverWidget.ts b/src/vs/workbench/services/hover/browser/hoverWidget.ts
@@ -141,7 +141,7 @@ export class HoverWidget extends Widget {
 		// Get horizontal alignment and position
 		let targetLeft = this._target.x !== undefined ? this._target.x : Math.min(...targetBounds.map(e => e.left));
 		if (targetLeft + this._hover.containerDomNode.clientWidth >= document.documentElement.clientWidth) {
-			this._x = document.documentElement.clientWidth;
+			this._x = document.documentElement.clientWidth - 1;
 			this._hover.containerDomNode.classList.add('right-aligned');
 		} else {
 			this._x = targetLeft;
diff --git a/src/vs/workbench/services/hover/browser/media/hover.css b/src/vs/workbench/services/hover/browser/media/hover.css
@@ -18,6 +18,11 @@
 	color: #3794ff;
 }
 
+.monaco-workbench .workbench-hover.right-aligned {
+	/* The context view service wraps strangely when it's right up against the edge without this */
+	left: 1px;
+}
+
 .monaco-workbench .workbench-hover.right-aligned .hover-row.status-bar .actions {
 	flex-direction: row-reverse;
 }

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,11 @@`
`18`	`18`	`color: #3794ff;`
`19`	`19`	`}`
`20`	`20`
	`21`	`+.monaco-workbench .workbench-hover.right-aligned {`
	`22`	`+ /* The context view service wraps strangely when it's right up against the edge without this */`
	`23`	`+ left: 1px;`
	`24`	`+}`
	`25`	`+`
`21`	`26`	`.monaco-workbench .workbench-hover.right-aligned .hover-row.status-bar .actions {`
`22`	`27`	`flex-direction: row-reverse;`
`23`	`28`	`}`