Skip to content

Commit

Permalink
fix(material/tooltip): only expand hover area on side closest to trigger
Browse files Browse the repository at this point in the history
Fixes that we were adding an additional hover area on all sides of the tooltip, rather than only the side that is next to the trigger.

Fixes angular#26557.
  • Loading branch information
crisbeto committed Mar 14, 2023
1 parent 005036b commit aafebd9
Showing 1 changed file with 24 additions and 4 deletions.
28 changes: 24 additions & 4 deletions src/material/tooltip/tooltip.scss
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,32 @@
&::before {
$offset: -8px;
content: '';
top: $offset;
right: $offset;
bottom: $offset;
left: $offset;
top: 0;
right: 0;
bottom: 0;
left: 0;
z-index: -1;
position: absolute;

// Only set the offset on the side closest to the panel so we
// don't accidentally cover more content than we need to.
.mat-mdc-tooltip-panel-below & {
top: $offset;
}

.mat-mdc-tooltip-panel-above & {
bottom: $offset;
}

// Note that here we use left/right instead of before/after
// so that we don't have to add extra styles for RTL.
.mat-mdc-tooltip-panel-right & {
left: $offset;
}

.mat-mdc-tooltip-panel-left & {
right: $offset;
}
}

&._mat-animation-noopable {
Expand Down

0 comments on commit aafebd9

Please sign in to comment.