Skip to content

Commit 5f6c325

Browse files
Marcel PützDevtools-frontend LUCI CQ
authored andcommitted
Support minimal distance to the viewport for tooltips
Demo: https://screencast.googleplex.com/cast/NTgyMzAwNzIwMDExNjczNnw3MDc3MzExNi1mZQ Bug: 392078321 Change-Id: I8a904de15b317720b62795df78456ac7656faadb Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6235026 Reviewed-by: Kim-Anh Tran <[email protected]> Commit-Queue: Marcel Pütz <[email protected]> Reviewed-by: Benedikt Meurer <[email protected]>
1 parent 99841be commit 5f6c325

File tree

1 file changed

+20
-2
lines changed

1 file changed

+20
-2
lines changed

front_end/ui/components/tooltip/tooltip.css

Lines changed: 20 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
*/
66

77
:host {
8+
--tooltip-viewport-distance: var(--sys-size-5);
89
/* Reset the browser's default styles for [popover] elements. */
910
margin: 0;
1011
background: none;
@@ -16,6 +17,8 @@
1617
position-try-fallbacks: flip-inline, flip-block, flip-inline flip-block;
1718

1819
& .container {
20+
width: max-content;
21+
margin: var(--sys-size-2);
1922
color: var(--sys-color-inverse-on-surface);
2023
background-color: var(--sys-color-inverse-surface);
2124
box-shadow: var(--sys-elevation-level2);
@@ -26,16 +29,31 @@
2629

2730
:host([variant="rich"]) {
2831
justify-self: unset;
32+
margin: 0 var(--tooltip-viewport-distance) var(--tooltip-viewport-distance) 0;
2933
position-area: bottom span-right;
34+
position-try-fallbacks: --bottom-left, --top-right, --top-left;
3035

3136
& .container {
37+
margin-inline: 0;
38+
margin-block: var(--sys-size-3);
3239
color: var(--sys-color-on-surface);
3340
background-color: var(--sys-color-base-container-elevated);
3441
border-radius: var(--sys-shape-corner-small);
3542
padding: var(--sys-size-6) var(--sys-size-8);
3643
}
3744
}
3845

39-
.container {
40-
margin: var(--sys-size-2);
46+
@position-try --bottom-left {
47+
position-area: bottom span-left;
48+
margin: 0 0 var(--tooltip-viewport-distance) var(--tooltip-viewport-distance);
49+
}
50+
51+
@position-try --top-right {
52+
position-area: top span-right;
53+
margin: var(--tooltip-viewport-distance) var(--tooltip-viewport-distance) 0 0;
54+
}
55+
56+
@position-try --top-left {
57+
position-area: top span-left;
58+
margin: var(--tooltip-viewport-distance) 0 0 var(--tooltip-viewport-distance);
4159
}

0 commit comments

Comments
 (0)