Skip to content

Commit 14bfd39

Browse files
devvaannshabose
authored andcommitted
feat: make dirty indicator work in overflow menu
1 parent 9375dd9 commit 14bfd39

File tree

2 files changed

+25
-12
lines changed

2 files changed

+25
-12
lines changed

src/extensionsIntegrated/TabBar/overflow.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,8 @@ define(function (require, exports, module) {
113113
const dropdown = new DropdownButton.DropdownButton("", hiddenTabs, function (item, index) {
114114
const iconHtml = item.$icon[0].outerHTML; // the file icon
115115
const dirtyHtml = item.isDirty
116-
? '<span class="tab-dirty-icon">•</span>'
117-
: ''; // to display the dirty icon in the overflow menu
116+
? '<span class="tab-dirty-icon-overflow">•</span>'
117+
: '<span class="tab-dirty-icon-overflow empty"></span>'; // adding an empty span for better alignment
118118

119119
const closeIconHtml =
120120
`<span class="tab-close-icon-overflow" data-tab-path="${item.path}" data-tab-index="${index}">
@@ -125,12 +125,14 @@ define(function (require, exports, module) {
125125
return {
126126
html:
127127
`<div class="dropdown-tab-item" data-tab-path="${item.path}">
128-
<span class="tab-icon-container">${iconHtml}</span>
129-
<span class="tab-name-container">${item.name}</span>
130-
${dirtyHtml}
128+
<div class="tab-info-container">
129+
${dirtyHtml}
130+
<span class="tab-icon-container">${iconHtml}</span>
131+
<span class="tab-name-container">${item.name}</span>
132+
</div>
131133
${closeIconHtml}
132134
</div>`,
133-
enabled: true // make sure items are enabled
135+
enabled: true
134136
};
135137
});
136138

src/styles/Extn-TabBar.less

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -217,14 +217,20 @@
217217
.dropdown-tab-item {
218218
display: flex;
219219
align-items: center;
220-
justify-content: flex-start;
221220
cursor: pointer;
221+
justify-content: space-between;
222222
}
223223

224224
.dropdown-tab-item:hover {
225225
background-color: #2A3B50;
226226
}
227227

228+
.tab-info-container {
229+
display: flex;
230+
align-items: center;
231+
flex-grow: 1;
232+
}
233+
228234
.tab-icon-container {
229235
margin-right: 0.25rem;
230236
display: inline-flex;
@@ -237,13 +243,18 @@
237243
text-overflow: ellipsis;
238244
}
239245

240-
.tab-dirty-icon {
246+
.tab-dirty-icon-overflow {
241247
color: #8D8D8E;
242-
font-size: 1.6rem;
248+
font-size: 1.2rem;
249+
width: 1rem;
250+
display: inline-flex;
251+
align-items: center;
252+
justify-content: center;
243253
margin-right: 0.25rem;
244-
position: absolute;
245-
left: 0.3rem;
246-
top: 0.25rem;
254+
}
255+
256+
.tab-dirty-icon-overflow.empty {
257+
visibility: hidden;
247258
}
248259

249260
.tab-close-icon-overflow {

0 commit comments

Comments
 (0)