Skip to content

Commit 0e65f27

Browse files
mrleemurraylemurra_microsoft
andauthored
Enhance layout consistency in sidebar and header components (#7633)
* Enhance layout spacing and padding in sidebar and header components * Update sidebar label class and adjust margin for improved layout consistency --------- Co-authored-by: lemurra_microsoft <Lee.Murray@microsoft.com>
1 parent 07b51fa commit 0e65f27

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

webviews/components/sidebar.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,7 @@ function CollapsedLabel(props: PullRequest) {
324324
return (
325325
<span
326326
key={getKey(item)}
327-
className="pill-item"
327+
className="pill-item label"
328328
style={{
329329
backgroundColor: color.backgroundColor,
330330
color: color.textColor,

webviews/editorWebview/index.css

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ textarea:focus,
5050
.title {
5151
display: flex;
5252
align-items: flex-start;
53-
margin: 20px 0;
53+
margin: 20px 0 24px;
5454
padding-bottom: 24px;
5555
border-bottom: 1px solid var(--vscode-list-inactiveSelectionBackground);
5656
}
@@ -555,6 +555,7 @@ small-button {
555555
.header-actions {
556556
display: flex;
557557
gap: 8px;
558+
padding-top: 4px;
558559
}
559560

560561
.header-actions>div:first-of-type {
@@ -608,7 +609,7 @@ small-button {
608609
}
609610

610611
.section {
611-
padding-bottom: 14px;
612+
padding-bottom: 16px;
612613
border-bottom: 1px solid var(--vscode-editorWidget-border);
613614
display: flex;
614615
flex-direction: column;
@@ -1237,6 +1238,7 @@ code {
12371238
#sidebar {
12381239
display: grid;
12391240
column-gap: 20px;
1241+
row-gap: 12px;
12401242
grid-template-columns: calc(50% - 10px) calc(50% - 10px);
12411243
padding: 0;
12421244
}
@@ -1483,20 +1485,20 @@ svg.octicon path {
14831485
.collapsible-sidebar {
14841486
border-top: 1px solid var(--vscode-editorWidget-border);
14851487
border-bottom: 1px solid var(--vscode-editorWidget-border);
1486-
margin-bottom: 16px;
1488+
margin-bottom: 24px;
14871489
}
14881490

14891491
.collapsible-sidebar-header {
14901492
display: flex;
14911493
align-items: center;
14921494
cursor: pointer;
1493-
padding: 16px 0px 7px;
1495+
padding: 16px 0px 8px;
14941496
user-select: none;
14951497
outline: none;
14961498
}
14971499

14981500
.collapsible-sidebar-header.expanded {
1499-
padding: 7px 0px;
1501+
padding: 8px 0px;
15001502
}
15011503

15021504
.collapsible-sidebar-header:focus {
@@ -1509,18 +1511,18 @@ svg.octicon path {
15091511
}
15101512

15111513
.collapsible-sidebar-content {
1512-
padding-bottom: 6px;
1514+
padding-bottom: 16px;
15131515
}
15141516

15151517
.collapsed-label {
1516-
gap: 6px;
1518+
gap: 8px;
15171519
display: grid;
15181520
grid-template-columns: 1fr 1fr;
1519-
grid-gap: 6px 12px;
1521+
grid-gap: 8px 20px;
15201522
}
15211523

15221524
.collapsed-section {
1523-
gap: 2px;
1525+
gap: 8px;
15241526
display: inline-flex;
15251527
align-items: center;
15261528
min-width: 0;
@@ -1547,8 +1549,6 @@ svg.octicon path {
15471549
flex-shrink: 0;
15481550
white-space: nowrap;
15491551
border-radius: 20px;
1550-
padding: 0px 6px;
1551-
font-size: 13px;
15521552
margin-right: 2px;
15531553
}
15541554

@@ -1570,7 +1570,8 @@ svg.octicon path {
15701570
}
15711571

15721572
.collapsible-label-see-more {
1573-
padding-bottom: 8px;
1573+
padding-bottom: 16px;
15741574
display: block;
1575-
font-size: 12px;
1575+
font-size: 13px;
1576+
cursor: pointer;
15761577
}

0 commit comments

Comments
 (0)