Skip to content

Commit caccd34

Browse files
committed
fix(web): flatten active session chrome
1 parent aeb38c5 commit caccd34

2 files changed

Lines changed: 23 additions & 3 deletions

File tree

packages/web/src/styles/components.css

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14612,7 +14612,7 @@ body.is-dragging-pane .session-action-btn-drag {
1461214612

1461314613
.session-card.session-card--active {
1461414614
background: var(--workspace-session-active-surface);
14615-
box-shadow: inset 0 0 0 1px var(--component-mix-border-focus-84pct-transparent);
14615+
box-shadow: none;
1461614616
}
1461714617

1461814618
.session-header,
@@ -14626,6 +14626,7 @@ body.is-dragging-pane .session-action-btn-drag {
1462614626

1462714627
.session-card.session-card--active > .panel-header,
1462814628
.session-card.session-card--active .session-header {
14629+
border-top: 1px solid var(--component-mix-border-focus-76pct-transparent);
1462914630
border-bottom-color: var(--component-mix-border-focus-76pct-transparent);
1463014631
}
1463114632

@@ -14789,9 +14790,14 @@ body.is-dragging-pane .session-action-btn-drag {
1478914790

1479014791
.mobile-shell__agent-stage .session-card.session-card--active {
1479114792
background: var(--workspace-session-mobile-active-surface);
14793+
box-shadow: none;
1479214794
backdrop-filter: var(--material-backdrop-filter);
1479314795
}
1479414796

14797+
.mobile-shell__agent-stage > .session-card.session-card--active > .panel-header {
14798+
border-top: 1px solid var(--component-mix-border-focus-76pct-transparent);
14799+
}
14800+
1479514801
@media (max-width: 899px), (pointer: coarse) {
1479614802
.mobile-shell {
1479714803
--mobile-safe-top: env(safe-area-inset-top, 0px);

packages/web/src/styles/components.theme.test.ts

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1147,9 +1147,9 @@ describe("components.css theme-sensitive surfaces", () => {
11471147
expect(sessionCard).toContain("backdrop-filter: var(--material-backdrop-filter)");
11481148
expect(activeSessionCard).toContain("background: var(--workspace-session-active-surface)");
11491149
expect(activeSessionCard).not.toContain("background: var(--bg-active)");
1150-
expect(activeSessionCard).toContain("box-shadow: inset 0 0 0 1px");
1151-
expect(activeSessionCard).toContain("var(--component-mix-border-focus-84pct-transparent)");
1150+
expect(activeSessionCard).toContain("box-shadow: none");
11521151
expect(activeSessionHeader).toContain("background: var(--workspace-session-header-surface)");
1152+
expect(activeSessionHeader).toContain("border-top:");
11531153
expect(activeSessionHeader).toContain("backdrop-filter: var(--material-backdrop-filter)");
11541154
expect(activeSessionHeader).not.toContain("var(--bg-active) 88%");
11551155
expect(activeSessionTitle).toContain("color: var(--text-primary)");
@@ -1538,6 +1538,10 @@ describe("components.css theme-sensitive surfaces", () => {
15381538
const xtermReplayCard = getLastRuleBlock(".xterm-replay-overlay__card");
15391539
const sessionProgress = getLastRuleBlock(".session-progress");
15401540
const sessionHeader = getLastRuleBlock(".session-header");
1541+
const activeSessionCard = getLastRuleBlock(".session-card.session-card--active");
1542+
const activeSessionHeader = getLastRuleBlock(
1543+
".session-card.session-card--active > .panel-header"
1544+
);
15411545
const supervisorCard = getLastRuleBlock(".supervisor-card");
15421546
const sessionHeaderLeft = getLastRuleBlock(".session-header-left");
15431547
const sessionHeaderCopyBlocks = getRuleBlocksFrom(stylesheet, ".session-header-copy");
@@ -1563,6 +1567,8 @@ describe("components.css theme-sensitive surfaces", () => {
15631567
expect(xtermReplayCard).toContain("border-radius: var(--terminal-local-overlay-radius)");
15641568
expect(sessionProgress).toContain("background: var(--state-info-bg)");
15651569
expect(sessionHeader).toContain("padding: var(--gap-tight) var(--inset-control-inline)");
1570+
expect(activeSessionCard).toContain("box-shadow: none");
1571+
expect(activeSessionHeader).toContain("border-top:");
15661572
expect(supervisorCard).toContain("background: var(--workspace-session-header-surface)");
15671573
expect(supervisorCard).toContain("backdrop-filter: var(--material-backdrop-filter)");
15681574
expect(sessionHeaderLeft).toContain("gap: var(--gap-default)");
@@ -3233,8 +3239,14 @@ describe("components.css theme-sensitive surfaces", () => {
32333239
".mobile-shell--landscape-compact .mobile-shell__viewport"
32343240
);
32353241
const sessionCard = getLastRuleBlock(".mobile-shell__agent-stage > .session-card");
3242+
const activeSessionCard = getLastRuleBlock(
3243+
".mobile-shell__agent-stage .session-card.session-card--active"
3244+
);
32363245
const progress = getLastRuleBlock(".mobile-shell__agent-stage .session-progress");
32373246
const header = getLastRuleBlock(".mobile-shell__agent-stage > .session-card > .panel-header");
3247+
const activeHeader = getLastRuleBlock(
3248+
".mobile-shell__agent-stage > .session-card.session-card--active > .panel-header"
3249+
);
32383250
const titleRow = getLastRuleBlock(".mobile-shell__agent-stage .session-title-row");
32393251
const headerRight = getLastRuleBlock(".mobile-shell__agent-stage .session-header-right");
32403252
const badges = getLastGroupedRuleBlock(
@@ -3252,9 +3264,11 @@ describe("components.css theme-sensitive surfaces", () => {
32523264
expect(content).toContain("gap: 4px");
32533265
expect(sessionCard).toContain("border-radius: 0");
32543266
expect(sessionCard).toContain("box-shadow: none");
3267+
expect(activeSessionCard).toContain("box-shadow: none");
32553268
expect(progress).toContain("display: none");
32563269
expect(header).toContain("padding: 4px");
32573270
expect(header).toContain("border-bottom:");
3271+
expect(activeHeader).toContain("border-top:");
32583272
expect(header).not.toContain("linear-gradient(");
32593273
expect(titleRow).toContain("gap: 6px");
32603274
expect(headerRight).toContain("max-width: 100%");

0 commit comments

Comments
 (0)