Skip to content

Commit 9a6ada9

Browse files
committed
Tighten supervisor dialog style contract
1 parent ca2e0b5 commit 9a6ada9

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

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

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -855,6 +855,12 @@ describe("components.css theme-sensitive surfaces", () => {
855855
const modalBody = getLastRuleBlock(".supervisor-dialog .modal-body");
856856
const formGroup = getLastRuleBlock(".supervisor-dialog .form-group");
857857
const intro = getLastRuleBlock(".supervisor-dialog-intro");
858+
const introEditTone = getLastRuleBlock(
859+
".supervisor-dialog--edit .supervisor-dialog-intro__icon"
860+
);
861+
const introDisableTone = getLastRuleBlock(
862+
".supervisor-dialog--disable .supervisor-dialog-intro__icon"
863+
);
858864
const introTitle = getLastRuleBlock(".supervisor-dialog-intro__title");
859865
const introDescription = getLastRuleBlock(".supervisor-dialog-intro__description");
860866
const compactInputGroup = getLastGroupedRuleBlock(
@@ -868,6 +874,14 @@ describe("components.css theme-sensitive surfaces", () => {
868874
expect(formGroup).toContain("gap: 6px");
869875
expect(intro).toContain("display: flex");
870876
expect(intro).toContain("padding: var(--sp-3)");
877+
expect(intro).toContain("border: 1px solid color-mix(in srgb, var(--border) 90%, transparent)");
878+
expect(intro).toContain(
879+
"background: color-mix(in srgb, var(--bg-surface) 88%, var(--bg-hover))"
880+
);
881+
expect(introEditTone).toContain("var(--icon-surface-info)");
882+
expect(introEditTone).toContain("var(--icon-info)");
883+
expect(introDisableTone).toContain("var(--icon-surface-error)");
884+
expect(introDisableTone).toContain("var(--icon-error)");
871885
expect(introTitle).toContain("font-size: var(--type-body-size)");
872886
expect(introTitle).toContain("line-height: var(--type-body-line-height)");
873887
expect(introDescription).toContain("font-size: var(--type-meta-size)");
@@ -876,6 +890,7 @@ describe("components.css theme-sensitive surfaces", () => {
876890
expect(compactInputGroup).toContain("line-height: var(--type-label-line-height)");
877891
expect(textarea).toContain("font-size: var(--type-code-inline-size)");
878892
expect(textarea).toContain("min-height: 104px");
893+
expect(textarea).toContain("color: var(--text-secondary)");
879894
expect(dangerCallout).toContain("gap: var(--sp-2)");
880895
expect(dangerCallout).toContain("padding: var(--sp-2) var(--sp-3)");
881896
expect(dangerCallout).toContain("border-left-width: 1px");

0 commit comments

Comments
 (0)