Skip to content

Commit 08ca9d5

Browse files
author
Noam Kfir
committed
Replace list item menu with single recalculate button to temporarily overcome css stacking context conflict
1 parent 544baa9 commit 08ca9d5

File tree

2 files changed

+39
-25
lines changed

2 files changed

+39
-25
lines changed

src/views-ui/codeAnalytics/main.scss

Lines changed: 22 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,13 @@ vscode-panel-view {
9696
margin: 0;
9797
}
9898

99-
.sf-menu {
100-
li {
99+
.list-item-menu-item {
100+
padding: 4px;
101+
margin: 0;
102+
}
103+
104+
ul.list-item-menu {
105+
li.list-item-menu {
101106
right: 0;
102107
min-width: unset;
103108
font-family: var(--font-family);
@@ -111,20 +116,21 @@ vscode-panel-view {
111116
cursor: pointer;
112117
}
113118

114-
li {
115-
color: var(--button-secondary-foreground);
116-
background: var(--button-secondary-background);
117-
}
118-
}
119-
120-
ul {
121-
left: unset;
122-
right: 0;
123-
min-width: unset;
124-
125-
li {
126-
color: var(--button-secondary-foreground);
127-
background: var(--button-secondary-background);
119+
// li {
120+
// color: var(--button-secondary-foreground);
121+
// background: var(--button-secondary-background);
122+
// }
123+
124+
ul {
125+
left: unset;
126+
right: 0;
127+
min-width: unset;
128+
z-index: 100;
129+
130+
li {
131+
color: var(--button-secondary-foreground);
132+
background: var(--button-secondary-background);
133+
}
128134
}
129135
}
130136
}

src/views/codeAnalytics/InsightListView/ItemRender/insightTemplateHtml.ts

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -77,15 +77,23 @@ export class InsightTemplateHtml
7777

7878
const threeDotImageUri = this._viewUris.image('three-dots.svg');
7979

80+
// let menuHtml = menuItems?.length > 0
81+
// ? `<ul class="list-item-menu sf-menu sf-js-enabled">
82+
// <li class="list-item-menu">
83+
// <img class="list-item-icon" src="${threeDotImageUri}" height="15">
84+
// <ul>
85+
// ${menuItems.join("")}
86+
// </ul>
87+
// </li>
88+
// </ul>`
89+
// : ``;
8090
let menuHtml = menuItems?.length > 0
81-
? `<ul class="list-item-menu sf-menu sf-js-enabled">
82-
<li class="list-item-menu-item">
83-
<img class="list-item-icon" src="${threeDotImageUri}" height="15">
84-
<ul>
85-
${menuItems.join("")}
86-
</ul>
87-
</li>
88-
</ul>`
91+
? `<div
92+
class="list-item-icon custom-start-date-recalculate-link codicon codicon-calendar"
93+
src="${threeDotImageUri}"
94+
title="Recalculate\n\nClick to recalculate this insight only using new data"
95+
height="15"
96+
></div>`
8997
: ``;
9098
// const menuItemsHtml = menuItems.length > 0
9199
// ? `<li class="list-item-menu-item">
@@ -126,8 +134,8 @@ export class InsightTemplateHtml
126134
${timeInfoHtml}
127135
${descriptionHtml}
128136
</div>
129-
${iconHtml}
130137
${menuHtml}
138+
${iconHtml}
131139
</div>
132140
${bodyHtml}
133141
${buttonsHtml}

0 commit comments

Comments
 (0)