Skip to content

Commit 426c78c

Browse files
committed
fix(web): unify overlay backdrops on shared token
1 parent d29a499 commit 426c78c

5 files changed

Lines changed: 16 additions & 8 deletions

File tree

packages/web/src/components/ui/drawer/index.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
display: flex;
77
justify-content: flex-end;
88
padding: var(--sp-4);
9-
background: var(--surface-overlay-backdrop);
9+
background: var(--overlay-backdrop);
1010
}
1111

1212
.panel,

packages/web/src/components/ui/local-overlay/index.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
align-items: center;
88
justify-content: center;
99
padding: var(--sp-4);
10-
background: var(--surface-overlay-backdrop);
10+
background: var(--overlay-backdrop);
1111
pointer-events: none;
1212
}
1313

packages/web/src/components/ui/modal/index.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
align-items: center;
88
justify-content: center;
99
padding: var(--sp-4);
10-
background: var(--surface-overlay-backdrop);
10+
background: var(--overlay-backdrop);
1111
}
1212

1313
.card,

packages/web/src/styles/components.css

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4457,7 +4457,7 @@ body.is-resizing-panels * {
44574457
.modal-overlay {
44584458
position: fixed;
44594459
inset: 0;
4460-
background: var(--surface-overlay-backdrop);
4460+
background: var(--overlay-backdrop);
44614461
display: flex;
44624462
align-items: center;
44634463
justify-content: center;
@@ -9936,7 +9936,7 @@ textarea.input {
99369936
.mobile-drawer-layer__backdrop {
99379937
position: absolute;
99389938
inset: 0;
9939-
background: rgba(3, 8, 12, 0.62);
9939+
background: var(--overlay-backdrop);
99409940
opacity: 0;
99419941
animation: mobile-shell-backdrop-enter var(--mobile-shell-motion-enter-duration) var(--ease-out)
99429942
forwards;

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

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -654,7 +654,7 @@ describe("components.css theme-sensitive surfaces", () => {
654654
"background: var(--state-error-bg)"
655655
);
656656

657-
expect(modalOverlay).toContain("background: var(--surface-overlay-backdrop)");
657+
expect(modalOverlay).toContain("background: var(--overlay-backdrop)");
658658
expect(
659659
getRuleBlocksFrom(modalStyles, ".card").some((block) =>
660660
block.includes("border: 1px solid var(--surface-overlay-border)")
@@ -676,7 +676,7 @@ describe("components.css theme-sensitive surfaces", () => {
676676
)
677677
).toBe(true);
678678

679-
expect(drawerBackdrop).toContain("background: var(--surface-overlay-backdrop)");
679+
expect(drawerBackdrop).toContain("background: var(--overlay-backdrop)");
680680
expect(
681681
getRuleBlocksFrom(drawerStylesheet, ".panel").some((block) =>
682682
block.includes("border: 1px solid var(--surface-overlay-border)")
@@ -698,7 +698,7 @@ describe("components.css theme-sensitive surfaces", () => {
698698
)
699699
).toBe(true);
700700

701-
expect(localOverlay).toContain("background: var(--surface-overlay-backdrop)");
701+
expect(localOverlay).toContain("background: var(--overlay-backdrop)");
702702
expect(localOverlayCard).toContain("border: 1px solid var(--surface-overlay-border)");
703703
expect(localOverlayCard).toContain("border-radius: var(--radius-local-overlay)");
704704
expect(localOverlayCard).toContain("background: var(--surface-overlay-bg)");
@@ -1041,6 +1041,14 @@ describe("components.css theme-sensitive surfaces", () => {
10411041
expect(sheetActions).toContain("gap: var(--gap-compact)");
10421042
});
10431043

1044+
it("keeps mobile sheet and drawer backdrops on the shared overlay backdrop token", () => {
1045+
const mobileSheetBackdrop = getLastRuleBlock(".mobile-sheet-layer__backdrop");
1046+
const mobileDrawerBackdrop = getLastRuleBlock(".mobile-drawer-layer__backdrop");
1047+
1048+
expect(mobileSheetBackdrop).toContain("background: var(--overlay-backdrop)");
1049+
expect(mobileDrawerBackdrop).toContain("background: var(--overlay-backdrop)");
1050+
});
1051+
10441052
it("keeps worktree state chips and mobile tabs on shared state and layer tokens", () => {
10451053
const cleanChip = getLastRuleBlock(".worktree-chip-status.worktree-clean");
10461054
const dirtyChip = getLastRuleBlock(".worktree-chip-status.worktree-dirty");

0 commit comments

Comments
 (0)