maxmustermann/vscode
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Hide terminal tab hover on mousein, unless actions

Browse source 
Fixes #122299
This commit is contained in:
Daniel Imms 2021-04-27 04:47:41 -07:00
parent eb141a8025
commit dbb1a58499
2 changed files with 10 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/vs/workbench/contrib/terminal/browser/terminalTabsWidget.ts
View file

@ -185,7 +185,8 @@ class TerminalTabsRenderer implements ITreeRenderer<ITerminalInstance, never, IT
showHover: options => {
return this._hoverService.showHover({
...options,
actions: context.hoverActions
actions: context.hoverActions,
hideOnHover: true
});
}
}

16
src/vs/workbench/services/hover/browser/hoverWidget.ts
View file

@ -143,17 +143,17 @@ export class HoverWidget extends Widget {
const mouseTrackerTargets = [...this._target.targetElements];
let hideOnHover: boolean;
if (options.hideOnHover === undefined) {
if (options.actions && options.actions.length > 0) {
// If there are actions, require hover so they can be accessed
hideOnHover = false;
} else {
if (options.actions && options.actions.length > 0) {
// If there are actions, require hover so they can be accessed
hideOnHover = false;
} else {
if (options.hideOnHover === undefined) {
// Defaults to true when string, false when markdown as it may contain links
hideOnHover = typeof options.text === 'string';
} else {
// It's set explicitly
hideOnHover = options.hideOnHover;
}
} else {
// It's set explicitly
hideOnHover = options.hideOnHover;
}
if (!hideOnHover) {
mouseTrackerTargets.push(this._hover.containerDomNode);