Skip to content

fix: pre-freeze audit fixes (rebased onto main) #121

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jackgranatowski merged 4 commits into from
May 27, 2026
Merged

fix: pre-freeze audit fixes (rebased onto main) #121

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a105e39
feat: apply PR #118 audit-fixes-pre-freeze changes to main
kiro-agent May 27, 2026
13746e6
fix: address review findings (motion docs, layout.md, dist rebuild)
kiro-agent May 27, 2026
bfb2802
chore: regenerate dist, docs, and registry; add unprefixed class trac…
kiro-agent May 27, 2026
1b9792a
test: add sf-icon--2xl to demo and update token snapshot
kiro-agent May 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion core/accessibility.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -161,15 +161,17 @@
top: var(--sf-space-m);
}


/* ----------------------------------------------------------
Focus-parent — an a11y *pattern*, not a state. Lives in the
accessibility layer (was previously in states.css; moved in
v0.3.0) so that all focus-related a11y rules share one home
and survive third-party resets at the same priority as the
focus-visible ring above. Cannot be hardened with !important
because the outline shorthand is consumer-customisable.
Canonical name: .sf-focus-parent (prefixed).
---------------------------------------------------------- */
.focus-parent:focus-within {
.sf-focus-parent:focus-within {
outline: var(--sf-focus-ring-width) var(--sf-focus-ring-style, solid) var(--sf-color-border--focus);
outline-offset: var(--sf-focus-ring-offset);
}
Expand Down
11 changes: 6 additions & 5 deletions core/layout.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -270,11 +270,12 @@
vertical-align: text-bottom;
fill: currentcolor;
}
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--2xl { --sf-icon-size: var(--sf-icon-2xl); }

/* Boxed variant — wraps the icon in a padded, bordered, optionally
coloured frame. Adds inline-/block-size on top of the naked icon
Expand Down
13 changes: 7 additions & 6 deletions core/tokens.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -561,14 +561,15 @@
--sf-icon-scale knob by design. clamp() is intentionally
absent here: making icons fluid INDEPENDENTLY of context
would break the relationship to surrounding text.
Capped at --xl (3em ≈ 48px on default text); above that the
Capped at --2xl (4em ≈ 64px on default text); above that the
element ceases to be an icon and becomes an illustration.
---------------------------------------------------------- */
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-2xl: 4em;


/* ----------------------------------------------------------
Expand Down
28 changes: 16 additions & 12 deletions dist/slashed.essential.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -586,14 +586,15 @@
--sf-icon-scale knob by design. clamp() is intentionally
absent here: making icons fluid INDEPENDENTLY of context
would break the relationship to surrounding text.
Capped at --xl (3em ≈ 48px on default text); above that the
Capped at --2xl (4em ≈ 64px on default text); above that the
element ceases to be an icon and becomes an illustration.
---------------------------------------------------------- */
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-2xl: 4em;


/* ----------------------------------------------------------
Expand Down Expand Up @@ -1948,11 +1949,12 @@
vertical-align: text-bottom;
fill: currentcolor;
}
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--2xl { --sf-icon-size: var(--sf-icon-2xl); }

/* Boxed variant — wraps the icon in a padded, bordered, optionally
coloured frame. Adds inline-/block-size on top of the naked icon
Expand Down Expand Up @@ -3139,15 +3141,17 @@
top: var(--sf-space-m);
}


/* ----------------------------------------------------------
Focus-parent — an a11y *pattern*, not a state. Lives in the
accessibility layer (was previously in states.css; moved in
v0.3.0) so that all focus-related a11y rules share one home
and survive third-party resets at the same priority as the
focus-visible ring above. Cannot be hardened with !important
because the outline shorthand is consumer-customisable.
Canonical name: .sf-focus-parent (prefixed).
---------------------------------------------------------- */
.focus-parent:focus-within {
.sf-focus-parent:focus-within {
outline: var(--sf-focus-ring-width) var(--sf-focus-ring-style, solid) var(--sf-color-border--focus);
outline-offset: var(--sf-focus-ring-offset);
}
Expand Down
28 changes: 16 additions & 12 deletions dist/slashed.essential.flat.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -568,14 +568,15 @@
--sf-icon-scale knob by design. clamp() is intentionally
absent here: making icons fluid INDEPENDENTLY of context
would break the relationship to surrounding text.
Capped at --xl (3em ≈ 48px on default text); above that the
Capped at --2xl (4em ≈ 64px on default text); above that the
element ceases to be an icon and becomes an illustration.
---------------------------------------------------------- */
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-2xl: 4em;


/* ----------------------------------------------------------
Expand Down Expand Up @@ -1914,11 +1915,12 @@ td, th {
vertical-align: text-bottom;
fill: currentcolor;
}
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--2xl { --sf-icon-size: var(--sf-icon-2xl); }

/* Boxed variant — wraps the icon in a padded, bordered, optionally
coloured frame. Adds inline-/block-size on top of the naked icon
Expand Down Expand Up @@ -3093,15 +3095,17 @@ td, th {
top: var(--sf-space-m);
}


/* ----------------------------------------------------------
Focus-parent — an a11y *pattern*, not a state. Lives in the
accessibility layer (was previously in states.css; moved in
v0.3.0) so that all focus-related a11y rules share one home
and survive third-party resets at the same priority as the
focus-visible ring above. Cannot be hardened with !important
because the outline shorthand is consumer-customisable.
Canonical name: .sf-focus-parent (prefixed).
---------------------------------------------------------- */
.focus-parent:focus-within {
.sf-focus-parent:focus-within {
outline: var(--sf-focus-ring-width) var(--sf-focus-ring-style, solid) var(--sf-color-border--focus);
outline-offset: var(--sf-focus-ring-offset);
}
Expand Down
2 changes: 1 addition & 1 deletion dist/slashed.essential.flat.min.css
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/slashed.essential.flat.min.css.map
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/slashed.essential.min.css
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/slashed.essential.min.css.map
View file
Open in desktop

Large diffs are not rendered by default.

34 changes: 20 additions & 14 deletions dist/slashed.full.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -586,14 +586,15 @@
--sf-icon-scale knob by design. clamp() is intentionally
absent here: making icons fluid INDEPENDENTLY of context
would break the relationship to surrounding text.
Capped at --xl (3em ≈ 48px on default text); above that the
Capped at --2xl (4em ≈ 64px on default text); above that the
element ceases to be an icon and becomes an illustration.
---------------------------------------------------------- */
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-2xl: 4em;


/* ----------------------------------------------------------
Expand Down Expand Up @@ -1948,11 +1949,12 @@
vertical-align: text-bottom;
fill: currentcolor;
}
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--2xl { --sf-icon-size: var(--sf-icon-2xl); }

/* Boxed variant — wraps the icon in a padded, bordered, optionally
coloured frame. Adds inline-/block-size on top of the naked icon
Expand Down Expand Up @@ -3139,15 +3141,17 @@
top: var(--sf-space-m);
}


/* ----------------------------------------------------------
Focus-parent — an a11y *pattern*, not a state. Lives in the
accessibility layer (was previously in states.css; moved in
v0.3.0) so that all focus-related a11y rules share one home
and survive third-party resets at the same priority as the
focus-visible ring above. Cannot be hardened with !important
because the outline shorthand is consumer-customisable.
Canonical name: .sf-focus-parent (prefixed).
---------------------------------------------------------- */
.focus-parent:focus-within {
.sf-focus-parent:focus-within {
outline: var(--sf-focus-ring-width) var(--sf-focus-ring-style, solid) var(--sf-color-border--focus);
outline-offset: var(--sf-focus-ring-offset);
}
Expand Down Expand Up @@ -3736,11 +3740,13 @@
full control via BEM components, skip this file entirely.

TOKENS CONSUMED:
--sf-radius-m, --sf-border-width-1, --sf-color-border,
--sf-radius-m, --sf-radius-full, --sf-border-width-1,
--sf-border-width-3, --sf-color-border,
--sf-color-border--focus, --sf-color-border--subtle,
--sf-color-surface, --sf-color-text, --sf-color-text--placeholder,
--sf-color-text--disabled, --sf-color-text--on-action,
--sf-color-bg--disabled, --sf-color-action, --sf-color-error,
--sf-color-bg--disabled, --sf-color-action, --sf-color-action-hover,
--sf-color-error,
--sf-field-border-color, --sf-opacity-disabled,
--sf-space-xs, --sf-space-s, --sf-space-m,
--sf-text-m, --sf-leading-normal, --sf-font-weight-heading,
Expand Down
34 changes: 20 additions & 14 deletions dist/slashed.full.flat.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -568,14 +568,15 @@
--sf-icon-scale knob by design. clamp() is intentionally
absent here: making icons fluid INDEPENDENTLY of context
would break the relationship to surrounding text.
Capped at --xl (3em ≈ 48px on default text); above that the
Capped at --2xl (4em ≈ 64px on default text); above that the
element ceases to be an icon and becomes an illustration.
---------------------------------------------------------- */
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-xs: 0.875em;
--sf-icon-s: 1em;
--sf-icon-m: 1.5em;
--sf-icon-l: 2em;
--sf-icon-xl: 3em;
--sf-icon-2xl: 4em;


/* ----------------------------------------------------------
Expand Down Expand Up @@ -1914,11 +1915,12 @@ td, th {
vertical-align: text-bottom;
fill: currentcolor;
}
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--xs { --sf-icon-size: var(--sf-icon-xs); }
.sf-icon--s { --sf-icon-size: var(--sf-icon-s); }
.sf-icon--m { --sf-icon-size: var(--sf-icon-m); }
.sf-icon--l { --sf-icon-size: var(--sf-icon-l); }
.sf-icon--xl { --sf-icon-size: var(--sf-icon-xl); }
.sf-icon--2xl { --sf-icon-size: var(--sf-icon-2xl); }

/* Boxed variant — wraps the icon in a padded, bordered, optionally
coloured frame. Adds inline-/block-size on top of the naked icon
Expand Down Expand Up @@ -3093,15 +3095,17 @@ td, th {
top: var(--sf-space-m);
}


/* ----------------------------------------------------------
Focus-parent — an a11y *pattern*, not a state. Lives in the
accessibility layer (was previously in states.css; moved in
v0.3.0) so that all focus-related a11y rules share one home
and survive third-party resets at the same priority as the
focus-visible ring above. Cannot be hardened with !important
because the outline shorthand is consumer-customisable.
Canonical name: .sf-focus-parent (prefixed).
---------------------------------------------------------- */
.focus-parent:focus-within {
.sf-focus-parent:focus-within {
outline: var(--sf-focus-ring-width) var(--sf-focus-ring-style, solid) var(--sf-color-border--focus);
outline-offset: var(--sf-focus-ring-offset);
}
Expand Down Expand Up @@ -3683,11 +3687,13 @@ td, th {
full control via BEM components, skip this file entirely.

TOKENS CONSUMED:
--sf-radius-m, --sf-border-width-1, --sf-color-border,
--sf-radius-m, --sf-radius-full, --sf-border-width-1,
--sf-border-width-3, --sf-color-border,
--sf-color-border--focus, --sf-color-border--subtle,
--sf-color-surface, --sf-color-text, --sf-color-text--placeholder,
--sf-color-text--disabled, --sf-color-text--on-action,
--sf-color-bg--disabled, --sf-color-action, --sf-color-error,
--sf-color-bg--disabled, --sf-color-action, --sf-color-action-hover,
--sf-color-error,
--sf-field-border-color, --sf-opacity-disabled,
--sf-space-xs, --sf-space-s, --sf-space-m,
--sf-text-m, --sf-leading-normal, --sf-font-weight-heading,
Expand Down
2 changes: 1 addition & 1 deletion dist/slashed.full.flat.min.css
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/slashed.full.flat.min.css.map
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/slashed.full.min.css
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/slashed.full.min.css.map
View file
Open in desktop

Large diffs are not rendered by default.

Loading