Skip to content

Commit 6631292

Browse files
lemurra_microsoftlemurra_microsoft
authored andcommitted
Refactor metadata display in GlobalSessionItem and SessionItem for improved clarity and styling consistency; adjust CSS for layout enhancements
1 parent b5a0ab6 commit 6631292

3 files changed

Lines changed: 25 additions & 14 deletions

File tree

webviews/dashboardView/components/GlobalSessionItem.tsx

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ export const GlobalSessionItem: React.FC<GlobalSessionItemProps> = ({
7171
</div>
7272
)}
7373
{(session.isTemporary || !session.isLocal) && (
74-
<div className="metadata-item">
74+
<div className="metadata-item status-and-date">
7575
{session.isTemporary ? (
7676
<span className="status-badge status-creating">
7777
<span className="codicon codicon-loading codicon-modifier-spin"></span>
@@ -88,11 +88,14 @@ export const GlobalSessionItem: React.FC<GlobalSessionItemProps> = ({
8888
{formatStatus(session.status, index)}
8989
</span>
9090
)}
91+
<span className="session-date" title={formatFullDateTime(session.dateCreated)}>{formatDate(session.dateCreated)}</span>
92+
</div>
93+
)}
94+
{session.isLocal && (
95+
<div className="metadata-item">
96+
<span title={formatFullDateTime(session.dateCreated)}>{formatDate(session.dateCreated)}</span>
9197
</div>
9298
)}
93-
<div className="metadata-item">
94-
<span title={formatFullDateTime(session.dateCreated)}>{formatDate(session.dateCreated)}</span>
95-
</div>
9699
{session.isLocal && session.branchName && (
97100
<div className="metadata-item">
98101
<span className="codicon codicon-git-branch"></span>

webviews/dashboardView/components/SessionItem.tsx

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ export const SessionItem: React.FC<SessionItemProps> = ({
6565
</div>
6666
<div className="item-metadata">
6767
{(session.isTemporary || !session.isLocal) && (
68-
<div className="metadata-item">
68+
<div className="metadata-item status-and-date">
6969
{session.isTemporary ? (
7070
<span className="status-badge status-creating">
7171
<span className="codicon codicon-loading codicon-modifier-spin"></span>
@@ -82,11 +82,14 @@ export const SessionItem: React.FC<SessionItemProps> = ({
8282
{formatStatus(session.status, index)}
8383
</span>
8484
)}
85+
<span className="session-date" title={formatFullDateTime(session.dateCreated)}>{formatDate(session.dateCreated)}</span>
86+
</div>
87+
)}
88+
{session.isLocal && (
89+
<div className="metadata-item">
90+
<span title={formatFullDateTime(session.dateCreated)}>{formatDate(session.dateCreated)}</span>
8591
</div>
8692
)}
87-
<div className="metadata-item">
88-
<span title={formatFullDateTime(session.dateCreated)}>{formatDate(session.dateCreated)}</span>
89-
</div>
9093
{session.isLocal && session.branchName && (
9194
<div className="metadata-item">
9295
<span className="codicon codicon-git-branch"></span>

webviews/dashboardView/index.css

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ body {
268268
overflow-y: auto;
269269
border: 1px solid var(--vscode-panel-border);
270270
border-radius: 4px;
271-
margin-top: 4px;
271+
margin-top: 8px;
272272
padding: 0;
273273
}
274274

@@ -401,7 +401,7 @@ body {
401401
justify-content: space-between;
402402
flex-wrap: wrap;
403403
gap: 8px;
404-
font-size: 12px;
404+
font-size: 11px;
405405
color: var(--vscode-descriptionForeground);
406406
}
407407

@@ -415,6 +415,15 @@ body {
415415
margin-right: 2px;
416416
}
417417

418+
.metadata-item.status-and-date {
419+
gap: 8px;
420+
}
421+
422+
.metadata-item.status-and-date .session-date {
423+
font-size: 11px;
424+
color: var(--vscode-descriptionForeground);
425+
}
426+
418427
.status-badge {
419428
padding: 2px 6px;
420429
border-radius: 3px;
@@ -739,10 +748,6 @@ body {
739748
font-style: italic;
740749
}
741750

742-
.quick-actions {
743-
margin-top: 8px;
744-
}
745-
746751
.quick-action-button {
747752
display: flex;
748753
align-items: center;

0 commit comments

Comments
 (0)