@lyeve-labs/ui-kit 0.13.0 → 0.13.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/components/SidebarNav.svelte +6 -1
- package/dist/index.d.ts +1 -1
- package/dist/index.js +1 -1
- package/dist/internal/field.d.ts +8 -0
- package/dist/internal/field.js +10 -0
- package/package.json +1 -1
|
@@ -209,10 +209,15 @@
|
|
|
209
209
|
that appears only once the group is open leaves the reference dangling
|
|
210
210
|
in the one state where a reader needs it to tell them what the button
|
|
211
211
|
will reveal.
|
|
212
|
+
|
|
213
|
+
It keeps the implicit list role. role="group" here overrode it, and a
|
|
214
|
+
listitem whose parent is not a list is an ARIA context error, so every
|
|
215
|
+
row of an open section came back as a serious axe violation. Grouping
|
|
216
|
+
semantics belong to a tree, where the parent is a treeitem; this is a
|
|
217
|
+
nav of plain links and the disclosure already says what it controls.
|
|
212
218
|
-->
|
|
213
219
|
<ul
|
|
214
220
|
id={listId(node.id)}
|
|
215
|
-
role="group"
|
|
216
221
|
class="{open && !node.disabled ? 'flex' : 'hidden'} mt-0.5 flex-col gap-0.5"
|
|
217
222
|
>
|
|
218
223
|
{#each children as child (child.id)}
|
package/dist/index.d.ts
CHANGED
|
@@ -84,4 +84,4 @@ export { openDialog, closeDialog, dismissDialog, dismissAllDialogs, confirm, set
|
|
|
84
84
|
export type { DialogOptions, DialogEntry, DialogSize } from './components/dialog/types.js';
|
|
85
85
|
export { cn, type ClassValue } from './utils/cn.js';
|
|
86
86
|
export { getTheme, setTheme, toggleTheme, themeBootScript, type Theme } from './utils/theme.js';
|
|
87
|
-
export declare const VERSION = "0.13.
|
|
87
|
+
export declare const VERSION = "0.13.1";
|
package/dist/index.js
CHANGED
|
@@ -90,4 +90,4 @@ export { getTheme, setTheme, toggleTheme, themeBootScript } from './utils/theme.
|
|
|
90
90
|
// ── Version ────────────────────────────────────────────────────────────────
|
|
91
91
|
// Generated from package.json by `pnpm version:sync`. Bump package.json, never
|
|
92
92
|
// this line; the build and the test suite fail when the two disagree.
|
|
93
|
-
export const VERSION = '0.13.
|
|
93
|
+
export const VERSION = '0.13.1';
|
package/dist/internal/field.d.ts
CHANGED
|
@@ -25,6 +25,14 @@ export declare const FIELD_ERROR = "text-xs text-danger";
|
|
|
25
25
|
* MultiSelect chip well - use `CONTROL_MULTILINE` instead and keep the token as
|
|
26
26
|
* a minimum.
|
|
27
27
|
*/
|
|
28
|
+
/**
|
|
29
|
+
* outline-none must be paired with a replacement. It was not here: theme.css
|
|
30
|
+
* declares a global :focus-visible outline of 2px solid brand, chosen for its
|
|
31
|
+
* contrast, and a utility beats the base layer, so these two cancelled the
|
|
32
|
+
* kit's own focus indicator for every text input, textarea, number and select.
|
|
33
|
+
* Focus was left as a 1px border-colour change. SidebarNav's buttons and
|
|
34
|
+
* CONTROL_SEGMENT already pair the two correctly; these did half of it.
|
|
35
|
+
*/
|
|
28
36
|
export declare const CONTROL_BASE: string;
|
|
29
37
|
/** As CONTROL_BASE, for controls whose height is driven by their content. */
|
|
30
38
|
export declare const CONTROL_MULTILINE: string;
|
package/dist/internal/field.js
CHANGED
|
@@ -25,12 +25,22 @@ export const FIELD_ERROR = 'text-xs text-danger';
|
|
|
25
25
|
* MultiSelect chip well - use `CONTROL_MULTILINE` instead and keep the token as
|
|
26
26
|
* a minimum.
|
|
27
27
|
*/
|
|
28
|
+
/**
|
|
29
|
+
* outline-none must be paired with a replacement. It was not here: theme.css
|
|
30
|
+
* declares a global :focus-visible outline of 2px solid brand, chosen for its
|
|
31
|
+
* contrast, and a utility beats the base layer, so these two cancelled the
|
|
32
|
+
* kit's own focus indicator for every text input, textarea, number and select.
|
|
33
|
+
* Focus was left as a 1px border-colour change. SidebarNav's buttons and
|
|
34
|
+
* CONTROL_SEGMENT already pair the two correctly; these did half of it.
|
|
35
|
+
*/
|
|
28
36
|
export const CONTROL_BASE = 'w-full h-control rounded-lg bg-surface-2 border px-3 text-sm text-fg ' +
|
|
29
37
|
'placeholder:text-faint transition-colors duration-150 outline-none ' +
|
|
38
|
+
'focus-visible:ring-2 focus-visible:ring-inset focus-visible:ring-brand ' +
|
|
30
39
|
'disabled:opacity-50 disabled:cursor-not-allowed';
|
|
31
40
|
/** As CONTROL_BASE, for controls whose height is driven by their content. */
|
|
32
41
|
export const CONTROL_MULTILINE = 'w-full min-h-control rounded-lg bg-surface-2 border px-3 py-2 text-sm text-fg ' +
|
|
33
42
|
'placeholder:text-faint transition-colors duration-150 outline-none ' +
|
|
43
|
+
'focus-visible:ring-2 focus-visible:ring-inset focus-visible:ring-brand ' +
|
|
34
44
|
'disabled:opacity-50 disabled:cursor-not-allowed';
|
|
35
45
|
/**
|
|
36
46
|
* The border a control carries at rest and on focus.
|
package/package.json
CHANGED