Merge pull request #97023 from microsoft/tyriar/r145_scroll

Tyriar · web-flow · commit c3b0ae556040 · 2020-05-05T13:27:41.000-07:00
Apply scroll area to hover content, not status bar
diff --git a/src/vs/workbench/contrib/terminal/browser/widgets/hoverWidget.ts b/src/vs/workbench/contrib/terminal/browser/widgets/hoverWidget.ts
@@ -93,7 +93,7 @@ export class HoverWidget extends Widget {
 			const actionsElement = $('div.actions');
 			this._actions.forEach(action => this._renderAction(actionsElement, action));
 			statusBarElement.appendChild(actionsElement);
-			this._domNode.appendChild(statusBarElement);
+			this._containerDomNode.appendChild(statusBarElement);
 		}
 
 		this._mouseTracker = new CompositeMouseTracker([this._containerDomNode, ..._target.targetElements]);

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ export class HoverWidget extends Widget {`
`93`	`93`	`const actionsElement = $('div.actions');`
`94`	`94`	`this._actions.forEach(action => this._renderAction(actionsElement, action));`
`95`	`95`	`statusBarElement.appendChild(actionsElement);`
`96`		`- this._domNode.appendChild(statusBarElement);`
	`96`	`+ this._containerDomNode.appendChild(statusBarElement);`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`this._mouseTracker = new CompositeMouseTracker([this._containerDomNode, ..._target.targetElements]);`