Skip to content

Commit 609ba7b

Browse files
devvaannshabose
authored andcommitted
chore: improve styling
1 parent bea78ec commit 609ba7b

File tree

2 files changed

+21
-20
lines changed

2 files changed

+21
-20
lines changed

src/extensionsIntegrated/TabBar/main.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,8 @@ define(function (require, exports, module) {
240240
$tabBar = $(TabBarHTML);
241241
// since we need to add the tab bar before the editor which has .not-editor class
242242
// we target the `.not-editor` class and add the tab bar before it
243-
$(".not-editor").before($tabBar);
244243
setTimeout(function () {
244+
$(".not-editor").before($tabBar);
245245
WorkspaceManager.recomputeLayout(true);
246246
}, 0);
247247

@@ -253,9 +253,9 @@ define(function (require, exports, module) {
253253
// here #editor-holder cannot be used as in split view, we only have one #editor-holder
254254
// so, right now we are using .not-editor. Maybe we need to look for some better selector
255255
// TODO: Fix bug where the tab bar gets hidden inside the editor in horizontal split
256-
$(".not-editor").eq(0).before($tabBar);
257-
$(".not-editor").eq(1).before($tabBar2);
258256
setTimeout(function () {
257+
$(".not-editor").eq(0).before($tabBar);
258+
$(".not-editor").eq(1).before($tabBar2);
259259
WorkspaceManager.recomputeLayout(true);
260260
}, 0);
261261
}

src/styles/Extn-TabBar.less

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
.phoenix-tab-bar {
1212
flex: 1;
13-
height: 2rem;
13+
height: 2.2rem;
1414
display: flex;
1515
overflow-x: auto;
1616
overflow-y: hidden;
@@ -94,11 +94,17 @@
9494

9595

9696
.tab-name {
97+
color: #aaa;
9798
display: inline-flex;
9899
align-items: center;
99-
font-size: 0.75rem;
100+
font-size: 0.9rem;
100101
letter-spacing: 0.4px;
101102
word-spacing: 0.75px;
103+
white-space: nowrap;
104+
}
105+
106+
.tab.active .tab-name {
107+
color: #dedede;
102108
}
103109

104110

@@ -126,7 +132,7 @@
126132
bottom: 0;
127133
left: 0;
128134
right: 0;
129-
height: 0.15rem;
135+
height: 0.12rem;
130136
background-color: #75BEFF;
131137
}
132138

@@ -135,26 +141,23 @@
135141
content: "";
136142
color: #8D8D8E;
137143
font-size: 1.6rem;
138-
margin-right: 0.25rem;
139144
position: absolute;
140-
left: 0.3rem;
141-
top: 0.25rem;
145+
left: 0.4rem;
146+
top: 0.3rem;
142147
}
143148

144149
.tab.dirty .tab-icon {
145-
margin-left: 1rem;
150+
margin-left: 0.8rem;
146151
}
147152

148153

149154
.tab-close {
150155
font-size: 0.75rem;
151-
font-weight: 300;
152-
padding: 0.2rem 0.5rem;
153-
margin-left: 0.5rem;
154-
margin-top: -0.1rem;
156+
padding: 0.08rem 0.4rem;
157+
margin-left: 0.25rem;
155158
color: #CCC;
156159
transition: all 0.2s ease;
157-
border-radius: 0.25rem;
160+
border-radius: 0.2rem;
158161
visibility: hidden;
159162
opacity: 0;
160163
pointer-events: none;
@@ -163,7 +166,7 @@
163166

164167
.tab:hover .tab-close,
165168
.tab.active .tab-close {
166-
visibility: visible;
169+
visibility: visible;;
167170
opacity: 1;
168171
pointer-events: auto;
169172
}
@@ -245,12 +248,10 @@
245248

246249
.tab-dirty-icon-overflow {
247250
color: #8D8D8E;
248-
font-size: 1.2rem;
251+
font-size: 1.6rem;
249252
width: 0.2rem;
250253
display: inline-flex;
251-
align-items: center;
252-
justify-content: center;
253-
margin-right: 0.25rem;
254+
margin: 0.125rem 0.6rem 0.4rem -0.2rem;
254255
}
255256

256257
.tab-dirty-icon-overflow.empty {

0 commit comments

Comments
 (0)