@crisp-ui-kit/crisp 0.55.2 → 0.55.4
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/cjs/components/columncalc/columncalc.recipe.js +20 -4
- package/dist/cjs/components/datatable/datatable.js +25 -16
- package/dist/cjs/components/datatable/datatable.recipe.js +46 -3
- package/dist/cjs/components/select/select.js +1 -1
- package/dist/cjs/components/table/table.recipe.js +20 -3
- package/dist/cjs/components/tabletoolbar/tabletoolbar.chrome.js +24 -3
- package/dist/columncalc.css +14 -4
- package/dist/columncalc.layered.css +14 -4
- package/dist/datatable.css +93 -79
- package/dist/datatable.layered.css +93 -79
- package/dist/esm/components/columncalc/columncalc.recipe.js +20 -4
- package/dist/esm/components/datatable/datatable.js +25 -16
- package/dist/esm/components/datatable/datatable.recipe.js +46 -3
- package/dist/esm/components/select/select.js +1 -1
- package/dist/esm/components/table/table.recipe.js +20 -3
- package/dist/esm/components/tabletoolbar/tabletoolbar.chrome.js +24 -3
- package/dist/styles.css +150 -95
- package/dist/styles.layered.css +150 -95
- package/dist/table.css +11 -3
- package/dist/table.layered.css +11 -3
- package/dist/tabletoolbar.css +32 -9
- package/dist/tabletoolbar.layered.css +32 -9
- package/dist/types/components/datatable/datatable.d.ts +17 -1
- package/dist/types/components/select/select.d.ts +5 -0
- package/package.json +1 -1
|
@@ -590,6 +590,23 @@ function DataReferenceEditor({ col, value, isPending, onCommit, onCancel, }) {
|
|
|
590
590
|
onCommit(v);
|
|
591
591
|
} }) }));
|
|
592
592
|
}
|
|
593
|
+
// crisp/#842, confirmed live -- Attio's own country-picker popover: each
|
|
594
|
+
// option row is plain text, name (opaque) + dial code (muted 63%), NO flag
|
|
595
|
+
// anywhere -- not in the option row, and not in the CLOSED trigger either,
|
|
596
|
+
// which shows only the bare 2-letter ISO code ("MN") + a chevron. That's a
|
|
597
|
+
// different, more compact string than the option row's own "{name}
|
|
598
|
+
// +{dialCode}", so this builds `label` (the option row) and `triggerLabel`
|
|
599
|
+
// (the closed trigger) as two different values on the same `SelectOption`
|
|
600
|
+
// rather than one flag+name+dialcode string doing both jobs -- the shape
|
|
601
|
+
// #765 originally shipped, unmeasured, and #842 corrected.
|
|
602
|
+
function buildCountryOptions() {
|
|
603
|
+
return COUNTRIES.map((c) => ({
|
|
604
|
+
value: c.iso2,
|
|
605
|
+
label: (_jsxs("span", { className: "crisp-datatable-country-option", children: [_jsx("span", { className: "crisp-datatable-country-name", children: c.name }), _jsxs("span", { className: "crisp-datatable-country-dial", children: ["+", c.dialCode] })] })),
|
|
606
|
+
triggerLabel: c.iso2,
|
|
607
|
+
keywords: `${c.name} ${c.iso2} ${c.dialCode}`,
|
|
608
|
+
}));
|
|
609
|
+
}
|
|
593
610
|
// crisp/#765, confirmed live -- Attio's own Phone Number editor: a
|
|
594
611
|
// country-code select (ghost trigger, "MN ▾"), then a static, non-editable
|
|
595
612
|
// "+{dialCode}" prefix, then a borderless digits field -- measured
|
|
@@ -597,20 +614,16 @@ function DataReferenceEditor({ col, value, isPending, onCommit, onCancel, }) {
|
|
|
597
614
|
// value already measured elsewhere in this file (14px in both cases).
|
|
598
615
|
// Reuses crisp's own searchable `Select` for the country list -- the same
|
|
599
616
|
// popover chrome every other picker in this file already uses -- rather
|
|
600
|
-
// than inventing new country-picker chrome; the
|
|
601
|
-
//
|
|
602
|
-
//
|
|
603
|
-
//
|
|
617
|
+
// than inventing new country-picker chrome; the panel/row chrome (radius,
|
|
618
|
+
// padding, row height) already lines up with `Select`'s own measured
|
|
619
|
+
// tokens (#842 re-confirmed this), so only the option content itself
|
|
620
|
+
// (`buildCountryOptions` above) needed a real fix.
|
|
604
621
|
function DataPhoneEditor({ col, value, isPending, onCommit, }) {
|
|
605
622
|
const parsed = parsePhoneValue(value);
|
|
606
623
|
const [iso2, setIso2] = React.useState(parsed?.iso2 ?? "US");
|
|
607
624
|
const [digits, setDigits] = React.useState(parsed?.digits ?? "");
|
|
608
625
|
const country = countryByIso2(iso2);
|
|
609
|
-
const countryOptions =
|
|
610
|
-
value: c.iso2,
|
|
611
|
-
label: `${flagEmoji(c.iso2)} ${c.name} +${c.dialCode}`,
|
|
612
|
-
keywords: `${c.name} ${c.iso2} ${c.dialCode}`,
|
|
613
|
-
}));
|
|
626
|
+
const countryOptions = buildCountryOptions();
|
|
614
627
|
const commit = (nextIso2, nextDigits) => onCommit(`${nextIso2}|${nextDigits}`);
|
|
615
628
|
return (_jsxs("span", { className: "crisp-datatable-phone-editor", children: [_jsx(Select, { variant: "ghost", searchable: true, "aria-label": `Edit ${col.key} country`, value: iso2, disabled: isPending, options: countryOptions, onValueChange: (v) => {
|
|
616
629
|
setIso2(v);
|
|
@@ -636,11 +649,7 @@ function DataPhoneMultiEditor({ col, value, isPending, onCommit, }) {
|
|
|
636
649
|
const committed = parsePhoneValues(value);
|
|
637
650
|
const [draftIso2, setDraftIso2] = React.useState("US");
|
|
638
651
|
const [draftDigits, setDraftDigits] = React.useState("");
|
|
639
|
-
const countryOptions =
|
|
640
|
-
value: c.iso2,
|
|
641
|
-
label: `${flagEmoji(c.iso2)} ${c.name} +${c.dialCode}`,
|
|
642
|
-
keywords: `${c.name} ${c.iso2} ${c.dialCode}`,
|
|
643
|
-
}));
|
|
652
|
+
const countryOptions = buildCountryOptions();
|
|
644
653
|
const join = (list) => list.map((p) => `${p.iso2}|${p.digits}`).join(";");
|
|
645
654
|
const removeAt = (i) => onCommit(join(committed.filter((_, idx) => idx !== i)));
|
|
646
655
|
const commitDraft = () => {
|
|
@@ -714,7 +723,7 @@ function DataDateTimeEditor({ col, value, isPending, onCommit, }) {
|
|
|
714
723
|
commit(date, t);
|
|
715
724
|
} })] }));
|
|
716
725
|
}
|
|
717
|
-
export function DataTable({ columns: columnsProp, data, selectable = false, selectedIds, onSelectionChange, selectAllLabel: selectAllLabelProp, selectRowLabel = (row) => `Select row ${row.id}`, selectionAnnouncement, columnMenu = false, onColumnsChange, columnOrder, onColumnOrderChange, groupBy, onRowsReorder, error, sort: sortProp, defaultSort, onSortChange, filterable = false, searchPlaceholder: searchPlaceholderProp, searchLabel: searchLabelProp, onEdit, onViewEditHistory, stickyFirst = true, stickyLast = false, scrollProps, locale: localeProp, avatarTone = "brand", surface, interactive = false, onRowClick, onAddRow, addRowLabel: addRowLabelProp, empty: emptyProp, showCount = true, countLabel, toolbar, height, className, reorderRowLabel: reorderRowLabelProp, openPreviewLabel: openPreviewLabelProp, sortAscendingLabel: sortAscendingLabelProp, sortDescendingLabel: sortDescendingLabelProp, moveColumnLeftLabel: moveColumnLeftLabelProp, moveColumnRightLabel: moveColumnRightLabelProp, editColumnLabel: editColumnLabelProp, hideColumnLabel: hideColumnLabelProp, renameColumnLabel: renameColumnLabelProp, viewSettingsLabel: viewSettingsLabelProp, columnsGroupLabel: columnsGroupLabelProp, cellMenuLabel: cellMenuLabelProp, copyLabel: copyLabelProp, pasteLabel: pasteLabelProp, viewEditHistoryLabel: viewEditHistoryLabelProp, clearValueLabel: clearValueLabelProp, ...rest }) {
|
|
726
|
+
export function DataTable({ columns: columnsProp, data, selectable = false, selectedIds, onSelectionChange, selectAllLabel: selectAllLabelProp, selectRowLabel = (row) => `Select row ${row.id}`, selectionAnnouncement, columnMenu = false, onColumnsChange, columnOrder, onColumnOrderChange, groupBy, onRowsReorder, error, loading = false, loadingLabel = "Loading...", sort: sortProp, defaultSort, onSortChange, filterable = false, searchPlaceholder: searchPlaceholderProp, searchLabel: searchLabelProp, onEdit, onViewEditHistory, stickyFirst = true, stickyLast = false, scrollProps, locale: localeProp, avatarTone = "brand", surface, interactive = false, onRowClick, onAddRow, addRowLabel: addRowLabelProp, empty: emptyProp, showCount = true, countLabel, toolbar, height, className, reorderRowLabel: reorderRowLabelProp, openPreviewLabel: openPreviewLabelProp, sortAscendingLabel: sortAscendingLabelProp, sortDescendingLabel: sortDescendingLabelProp, moveColumnLeftLabel: moveColumnLeftLabelProp, moveColumnRightLabel: moveColumnRightLabelProp, editColumnLabel: editColumnLabelProp, hideColumnLabel: hideColumnLabelProp, renameColumnLabel: renameColumnLabelProp, viewSettingsLabel: viewSettingsLabelProp, columnsGroupLabel: columnsGroupLabelProp, cellMenuLabel: cellMenuLabelProp, copyLabel: copyLabelProp, pasteLabel: pasteLabelProp, viewEditHistoryLabel: viewEditHistoryLabelProp, clearValueLabel: clearValueLabelProp, ...rest }) {
|
|
718
727
|
// crisp/#471: same resolution order as TableToolbar's own — an explicit
|
|
719
728
|
// prop wins, then the provider's translation of the built-in English
|
|
720
729
|
// default, then that default itself with no provider in the tree.
|
|
@@ -2139,7 +2148,7 @@ export function DataTable({ columns: columnsProp, data, selectable = false, sele
|
|
|
2139
2148
|
} })) : columnMenu && !isPrimary ? (_jsxs(Menu, { align: "start", trigger: _jsx("button", { type: "button", className: "crisp-datatable-sort", "data-sorted": sortDirByKey.get(col.key), "aria-label": `${typeof col.header === "string"
|
|
2140
2149
|
? col.header
|
|
2141
2150
|
: col.key} ${crispMessages["column options"] ?? "column options"}`, children: inner }), children: [_jsx(MenuItem, { onSelect: () => setSort({ key: col.key, dir: "asc" }), children: sortAscendingLabel }), _jsx(MenuItem, { onSelect: () => setSort({ key: col.key, dir: "desc" }), children: sortDescendingLabel }), col.managed === false ? null : (_jsxs(_Fragment, { children: [_jsx(MenuItem, { onSelect: () => moveColumn(col.key, -1), children: moveColumnLeftLabel }), _jsx(MenuItem, { onSelect: () => moveColumn(col.key, 1), children: moveColumnRightLabel }), _jsx(MenuItem, { onSelect: () => setRenaming(col.key), children: editColumnLabel }), _jsx(MenuItem, { onSelect: () => hideColumn(col.key), children: hideColumnLabel })] }))] })) : col.sortable && !isPrimary ? (_jsx("button", { type: "button", className: "crisp-datatable-sort", "data-sorted": sortDirByKey.get(col.key), onClick: () => cycleSort(col.key), children: inner })) : (inner), col.readOnly && (_jsx(Lock, { className: "crisp-datatable-readonly", size: 12, strokeWidth: 1.75, "aria-hidden": "true" })), col.resizable !== false && (_jsx("span", { className: "crisp-datatable-resize", "data-resizing": resizingKey === col.key ? "" : undefined, onPointerDown: (e) => startResize(col, e), onPointerMove: moveResize, onPointerUp: endResize }))] }, col.key));
|
|
2142
|
-
})] }) }), _jsx(TableBody, { children: error ? (_jsx(TableRow, { children: _jsx(TableCell, { colSpan: totalCols, children: _jsx("div", { className: "crisp-datatable-empty", children: error }) }) })) : rows.length === 0 ? (_jsx(TableRow, { children: _jsx(TableCell, { colSpan: totalCols, children: _jsx("div", { className: "crisp-datatable-empty", children: empty }) }) })) : (_jsxs(_Fragment, { children: [topSpacerHeight > 0 && (_jsx("tr", { "aria-hidden": "true", children: _jsx("td", { colSpan: totalCols, style: {
|
|
2151
|
+
})] }) }), _jsx(TableBody, { children: loading ? (_jsx(TableRow, { children: _jsx(TableCell, { colSpan: totalCols, children: _jsxs("div", { className: "crisp-datatable-loading", children: [_jsx(Spinner, { size: 16 }), loadingLabel] }) }) })) : error ? (_jsx(TableRow, { children: _jsx(TableCell, { colSpan: totalCols, children: _jsx("div", { className: "crisp-datatable-empty", children: error }) }) })) : rows.length === 0 ? (_jsx(TableRow, { children: _jsx(TableCell, { colSpan: totalCols, children: _jsx("div", { className: "crisp-datatable-empty", children: empty }) }) })) : (_jsxs(_Fragment, { children: [topSpacerHeight > 0 && (_jsx("tr", { "aria-hidden": "true", children: _jsx("td", { colSpan: totalCols, style: {
|
|
2143
2152
|
height: topSpacerHeight,
|
|
2144
2153
|
padding: 0,
|
|
2145
2154
|
border: "none",
|
|
@@ -65,8 +65,17 @@ const config = defineRecipe({
|
|
|
65
65
|
in the whole product and making every interactive column's button
|
|
66
66
|
double it to 24px. Scoping through \`.crisp-table\` adds a second
|
|
67
67
|
class to the selector (0,2,1 beats 0,1,1) so this can never lose a
|
|
68
|
-
specificity tie to source order again.
|
|
69
|
-
|
|
68
|
+
specificity tie to source order again.
|
|
69
|
+
\`thead th\`, not a bare \`th\` (crisp#829): #779 made the primary
|
|
70
|
+
column's BODY cell a real \`<th scope="row">\` for a11y, and a bare
|
|
71
|
+
\`th\` here matches that one too, zeroing an inset nothing else
|
|
72
|
+
supplies -- the primary column's text sat flush left, one column's
|
|
73
|
+
cell no longer matching its own header or any sibling \`<td>\`. Only
|
|
74
|
+
the header row's own \`<th>\` needs zero padding (its sort BUTTON
|
|
75
|
+
owns the inset); the body row's \`<th scope="row">\` has no button,
|
|
76
|
+
and needs the SAME base 12px (table.recipe.ts, #465) every other
|
|
77
|
+
cell already gets. */
|
|
78
|
+
& .crisp-table thead th { padding: 0; }
|
|
70
79
|
/* Same wrapper trap as AUTHORING.md Lesson 40 (ColumnCalc's summary trigger):
|
|
71
80
|
Menu wraps its trigger in a flex div that sizes to its content, so a
|
|
72
81
|
long header grew past the column instead of filling it -- with nothing
|
|
@@ -123,7 +132,22 @@ const config = defineRecipe({
|
|
|
123
132
|
& .crisp-datatable-sort[data-sorted] .crisp-datatable-chevron { opacity: 1; color: ${c.fgPrimary}; }
|
|
124
133
|
/* sticky first column(s) + horizontal scroll shadow — RecordsTable's feel */
|
|
125
134
|
& th[data-sticky], & td[data-sticky] { position: sticky; z-index: 1; background: ${c.bgSurface}; }
|
|
126
|
-
|
|
135
|
+
/* \`thead th[data-sticky]\`, not a bare \`th[data-sticky]\` (crisp#848, same
|
|
136
|
+
bare-\`th\` family as #829/#833/#835): this z-index:3 was written for
|
|
137
|
+
the HEADER's own sticky corner cell (sticky column x sticky header),
|
|
138
|
+
meant to win over everything else. Unscoped, it also matched the
|
|
139
|
+
primary column's BODY cell -- a real \`<th scope="row" data-sticky>\`
|
|
140
|
+
since #779 -- giving every scrolled body row's own sticky cell a
|
|
141
|
+
HIGHER z-index than the sticky FOOTER's (2), so it painted over the
|
|
142
|
+
footer instead of the footer showing through. Confirmed live on a
|
|
143
|
+
real bounded-height, 25-row table (GRC's Risk register): scrolled
|
|
144
|
+
partway, \`elementFromPoint\` at the footer's own on-screen cell
|
|
145
|
+
resolved to a body row's \`<th>\`, not the footer's \`<td>\`, and that
|
|
146
|
+
\`<th>\`'s computed z-index read 3. Scoped to \`thead\`, a body row's
|
|
147
|
+
own sticky \`<th>\` now falls back to the base rule above (z-index 1,
|
|
148
|
+
correct -- the same as a plain sticky \`<td>\`), and the footer's own
|
|
149
|
+
explicit z-index below is unaffected either way. */
|
|
150
|
+
& thead th[data-sticky] { z-index: 3; }
|
|
127
151
|
/* #554: a footer sticky-column cell sits in the bottom-sticky row (table.recipe.ts's
|
|
128
152
|
\`tfoot td { z-index: 2 }\`), which \`td[data-sticky]\`'s own higher-specificity
|
|
129
153
|
\`z-index: 1\` would otherwise undercut -- keep it level with the row it's in. */
|
|
@@ -180,6 +204,16 @@ const config = defineRecipe({
|
|
|
180
204
|
& .crisp-datatable-refoption-avatar { flex-shrink: 0; }
|
|
181
205
|
& .crisp-datatable-refoption--create { color: ${c.fgMuted}; }
|
|
182
206
|
& .crisp-datatable-empty { padding: ${c.space["8"]} ${c.space["3"]}; text-align: center; color: ${c.fgMuted}; font-size: ${c.text.lg}; }
|
|
207
|
+
/* #826 -- the full-body loading state (a filter/sort re-query, or a
|
|
208
|
+
fresh page boot), measured live on Attio's Companies grid on both
|
|
209
|
+
triggers: icon+text centred in the row-body area, 10px gap, 14px/500
|
|
210
|
+
weight/20px line-height, both icon and text in the SAME muted
|
|
211
|
+
rgba(0,0,0,0.63) -- the same color-mix figure this file already uses
|
|
212
|
+
for other muted text (e.g. -phone-dial). Distinct from the cold
|
|
213
|
+
in-place row load (blank, no indicator) and the warm-navigation
|
|
214
|
+
per-cell shimmer (#599) -- neither of those swaps the whole body. */
|
|
215
|
+
& .crisp-datatable-loading { display: flex; align-items: center; justify-content: center; gap: 10px; padding: ${c.space["8"]} ${c.space["3"]}; color: color-mix(in srgb, ${c.fgPrimary} 63%, transparent); font-size: ${c.text.lg}; font-weight: ${c.weight.medium}; line-height: 20px; }
|
|
216
|
+
& .crisp-datatable-loading .crisp-spinner { border-top-color: color-mix(in srgb, ${c.fgPrimary} 63%, transparent); }
|
|
183
217
|
/* #579: click-to-expand overlay for a truncated cell -- a read-only,
|
|
184
218
|
full-value card, settled live as click (not Attio's usual
|
|
185
219
|
hover-expand) and measured position: absolute, unlike this file's
|
|
@@ -291,6 +325,15 @@ const config = defineRecipe({
|
|
|
291
325
|
& .crisp-datatable-phone-dial { color: color-mix(in srgb, ${c.fgPrimary} 63%, transparent); }
|
|
292
326
|
& .crisp-datatable-phone-editor { display: flex; align-items: center; gap: ${c.space["1"]}; height: ${c.size.tableRow}; padding: 0 ${c.space["2"]}; }
|
|
293
327
|
& .crisp-datatable-phone-editor input { flex: 1; min-width: 0; }
|
|
328
|
+
/* country-picker option row (#842) -- confirmed live: Attio's own popover
|
|
329
|
+
is plain text, no flag anywhere (not the option row, not the closed
|
|
330
|
+
trigger), name opaque + dial code muted rgba(0,0,0,0.63) -- the SAME
|
|
331
|
+
color-mix figure as -phone-dial above. The internal name/dial gap
|
|
332
|
+
wasn't independently measured this pass (Attio's own two <div>s gave
|
|
333
|
+
no exact number); reused space["1"], the same gap -phone already uses
|
|
334
|
+
for its own flag/dial/digits row. */
|
|
335
|
+
& .crisp-datatable-country-option { display: flex; align-items: center; gap: ${c.space["1"]}; }
|
|
336
|
+
& .crisp-datatable-country-dial { color: color-mix(in srgb, ${c.fgPrimary} 63%, transparent); }
|
|
294
337
|
/* multi-value phone (#816) -- confirmed live (a real custom "Multiple
|
|
295
338
|
values" Phone Number attribute, and the built-in People "Phone
|
|
296
339
|
numbers" field): the collapsed cell is a 4px-gap flex row of real
|
|
@@ -168,7 +168,7 @@ export function Select({ options, value, defaultValue, onValueChange, name, requ
|
|
|
168
168
|
return (_jsxs("div", { ref: rootRef, className: selectRecipe({ variant, width, invalid, className }), ...rest, children: [name && (selected || required) && (_jsx("input", { className: "crisp-formvalue", style: formValueStyle, name: name, value: selected ?? "", required: required, tabIndex: -1, "aria-hidden": "true",
|
|
169
169
|
// controlled by the listbox above; `readOnly` would bar it from
|
|
170
170
|
// validation, which is the only reason this input is not hidden
|
|
171
|
-
onChange: () => { } })), _jsxs("button", { ref: triggerRef, type: "button", id: id, role: "combobox", "aria-haspopup": "listbox", "aria-expanded": open, "aria-controls": open ? listboxId : undefined, "aria-activedescendant": searchable ? undefined : activeId, "aria-label": ariaLabel, "aria-invalid": invalid === true || undefined, disabled: disabled, "data-open": open ? "" : undefined, className: "crisp-select-trigger", onClick: () => (open ? setOpen(false) : openMenu()), onKeyDown: onTriggerKeyDown, children: [_jsx("span", { className: "crisp-select-value", children: selectedOption ? (_jsxs(_Fragment, { children: [selectedOption.color && (_jsx("span", { className: "crisp-select-option-dot", style: { background: selectedOption.color } })), selectedOption.label] })) : (_jsx("span", { className: "crisp-select-placeholder", children: placeholder })) }), _jsx(ChevronDown, { className: "crisp-select-chevron", size: 16, strokeWidth: 1.75 })] }), open && (_jsx(FloatingPortal, { sourceRef: rootRef, children: _jsxs("div", { ref: listboxRef, id: listboxId, className: "crisp-select-listbox", role: "listbox", "aria-label": ariaLabel, "data-up": up ? "" : undefined, style: pos, children: [searchable && (_jsxs("div", { className: "crisp-select-search", children: [_jsx(Search, { size: 15, strokeWidth: 1.75 }), _jsx("input", { ref: searchRef, type: "text", placeholder: searchPlaceholder, "aria-label": searchPlaceholder, "aria-controls": listboxId, "aria-activedescendant": activeId, value: query, onChange: (e) => {
|
|
171
|
+
onChange: () => { } })), _jsxs("button", { ref: triggerRef, type: "button", id: id, role: "combobox", "aria-haspopup": "listbox", "aria-expanded": open, "aria-controls": open ? listboxId : undefined, "aria-activedescendant": searchable ? undefined : activeId, "aria-label": ariaLabel, "aria-invalid": invalid === true || undefined, disabled: disabled, "data-open": open ? "" : undefined, className: "crisp-select-trigger", onClick: () => (open ? setOpen(false) : openMenu()), onKeyDown: onTriggerKeyDown, children: [_jsx("span", { className: "crisp-select-value", children: selectedOption ? (_jsxs(_Fragment, { children: [selectedOption.color && (_jsx("span", { className: "crisp-select-option-dot", style: { background: selectedOption.color } })), selectedOption.triggerLabel ?? selectedOption.label] })) : (_jsx("span", { className: "crisp-select-placeholder", children: placeholder })) }), _jsx(ChevronDown, { className: "crisp-select-chevron", size: 16, strokeWidth: 1.75 })] }), open && (_jsx(FloatingPortal, { sourceRef: rootRef, children: _jsxs("div", { ref: listboxRef, id: listboxId, className: "crisp-select-listbox", role: "listbox", "aria-label": ariaLabel, "data-up": up ? "" : undefined, style: pos, children: [searchable && (_jsxs("div", { className: "crisp-select-search", children: [_jsx(Search, { size: 15, strokeWidth: 1.75 }), _jsx("input", { ref: searchRef, type: "text", placeholder: searchPlaceholder, "aria-label": searchPlaceholder, "aria-controls": listboxId, "aria-activedescendant": activeId, value: query, onChange: (e) => {
|
|
172
172
|
setQuery(e.target.value);
|
|
173
173
|
setHighlight(0);
|
|
174
174
|
onSearchChange?.(e.target.value);
|
|
@@ -25,8 +25,25 @@ const config = defineRecipe({
|
|
|
25
25
|
/* #554: sticky on the vertical axis, mirroring RecordsTable's own
|
|
26
26
|
\`.crisp-recordstable-th\`. z-index 2 sits above a plain body cell (auto)
|
|
27
27
|
and below a sticky-column corner cell (DataTable's \`th[data-sticky]\`,
|
|
28
|
-
z-index 3), so the corner always wins.
|
|
29
|
-
|
|
28
|
+
z-index 3), so the corner always wins.
|
|
29
|
+
\`thead th\`, not a bare \`th\` (crisp#833/#835): DataTable's primary
|
|
30
|
+
column renders its BODY cell as a real \`<th scope="row">\` (#779, for
|
|
31
|
+
a11y), and a bare \`th\` here matched that one too -- every property in
|
|
32
|
+
this rule leaked onto it: \`height: tableHeader\` (40px) stretched the
|
|
33
|
+
whole body row to 40px instead of 36 (#833, confirmed live on the
|
|
34
|
+
showcase, every td in the row equalized to the one th); \`color:
|
|
35
|
+
color-mix(...63%...)\` muted the primary column's own text below every
|
|
36
|
+
sibling \`<td>\`'s full-opacity color (#835, re-measured live against
|
|
37
|
+
Attio's own header labels -- including a genuinely computed one,
|
|
38
|
+
"Twitter follower count" -- all fully opaque \`rgb(16,17,18)\`, no
|
|
39
|
+
muting at all, so this drops the color-mix entirely rather than
|
|
40
|
+
narrowing it); \`position:sticky; top:0\` tried to stick a BODY row
|
|
41
|
+
to the scroll container's top; the extra \`border-top\` doubled that
|
|
42
|
+
row's own grid line. \`thead th\` simply never matches
|
|
43
|
+
\`tbody th[scope=row]\` at all, so none of this reaches it; the base
|
|
44
|
+
\`& th, & td\` rule above (36px, one border-bottom, full opacity via
|
|
45
|
+
\`& table\`'s own color) applies to it like every other cell. */
|
|
46
|
+
& thead th { position: sticky; top: 0; z-index: 2; height: ${c.size.tableHeader}; background: ${c.bgSurface}; border-top: 1px solid ${c.borderSubtle}; font-size: ${c.text.lg}; font-weight: ${c.weight.medium}; letter-spacing: -0.14px; color: ${c.fgPrimary}; user-select: none; }
|
|
30
47
|
& td { color: ${c.fgPrimary}; background: ${c.bgSurface}; }
|
|
31
48
|
& th:last-child, & td:last-child { border-inline-end: none; }
|
|
32
49
|
& tbody tr:last-child td { border-bottom: none; }
|
|
@@ -53,7 +70,7 @@ const config = defineRecipe({
|
|
|
53
70
|
// bottom one: the container's edge wins, cells never double it.
|
|
54
71
|
surface: {
|
|
55
72
|
card: `border: 1px solid ${c.borderSubtle}; border-radius: ${c.radius.xl}; overflow: hidden;
|
|
56
|
-
& th { border-top: none; }
|
|
73
|
+
& thead th { border-top: none; }
|
|
57
74
|
& tfoot td { border-bottom: none; }`,
|
|
58
75
|
bleed: `border: none; border-radius: 0;`,
|
|
59
76
|
},
|
|
@@ -50,7 +50,14 @@ export const toolbarChromeHead = (p) => ` & .${CLASS_PREFIX}${p}-toolbar { di
|
|
|
50
50
|
& .${CLASS_PREFIX}${p}-search svg { flex-shrink: 0; }
|
|
51
51
|
& .${CLASS_PREFIX}${p}-search input { border: none; outline: none; background: transparent; font-family: inherit; font-size: ${c.text.md}; color: ${c.fgPrimary}; width: 180px; }
|
|
52
52
|
& .${CLASS_PREFIX}${p}-search input::placeholder { color: ${c.fgMuted}; }
|
|
53
|
-
|
|
53
|
+
/* crisp/#830: every trigger label here (Sort, Filter, View settings,
|
|
54
|
+
Import/Export -- all compose this one shared .tool rule) read the
|
|
55
|
+
"md" text size, 13px. Re-measured on Attio's own trigger LEAF node
|
|
56
|
+
(the button root itself inherits Attio's own 10px rem base and is a
|
|
57
|
+
red herring, the same trap the original "10px font" layout-audit
|
|
58
|
+
finding fell into): every label leaf is 14px/500/lh20, the "lg" text
|
|
59
|
+
size. */
|
|
60
|
+
& .${CLASS_PREFIX}${p}-tool { display: inline-flex; align-items: center; gap: ${c.space["1"]}; height: ${c.size.row}; margin: 0; padding: 0 ${c.space["2"]}; border: none; border-radius: ${c.radius.md}; background: ${c.bgSurface}; box-shadow: inset 0 0 0 1px ${c.borderDivider}; color: ${c.fgPrimary}; font-family: inherit; font-size: ${c.text.lg}; font-weight: ${c.weight.medium}; white-space: nowrap; cursor: pointer; transition: background-color .12s; }
|
|
54
61
|
& .${CLASS_PREFIX}${p}-tool:hover { background: ${c.bgRaised}; }
|
|
55
62
|
& .${CLASS_PREFIX}${p}-tool svg { flex-shrink: 0; color: ${c.fgMuted}; }
|
|
56
63
|
/* crisp/#772: re-measured live on Sort with an actual sort applied --
|
|
@@ -75,8 +82,22 @@ export const toolbarChromeHead = (p) => ` & .${CLASS_PREFIX}${p}-toolbar { di
|
|
|
75
82
|
same reasoning --ghost above already uses. Only wired into
|
|
76
83
|
TableToolbar itself (tabletoolbar.tsx) this pass -- RecordsTable's
|
|
77
84
|
own Sort/Filter triggers share this same chrome function but are
|
|
78
|
-
deprecated, not touched here.
|
|
79
|
-
|
|
85
|
+
deprecated, not touched here.
|
|
86
|
+
crisp/#844, correcting #831: the label colour was added here later,
|
|
87
|
+
muting only the FILTER trigger (via a "--filter" modifier), on the
|
|
88
|
+
assumption that was a Filter-specific tone distinct from Sort's.
|
|
89
|
+
Re-measured live and that premise was wrong -- Sort's own resting
|
|
90
|
+
label ("Sort", no condition set) is the exact SAME muted rgba(0,0,0,.5)
|
|
91
|
+
as Filter's, not full-strength. (The rgba(0,0,0,.63) that looked like
|
|
92
|
+
"Sort's colour" in an earlier pass is a different thing entirely: the
|
|
93
|
+
"Sorted by" PREFIX word inside an ACTIVE sort chip, #788 -- a
|
|
94
|
+
compound-label treatment that only exists once a sort is applied, not
|
|
95
|
+
a resting-state colour.) --dashed is currently only ever applied to
|
|
96
|
+
Sort's and Filter's own empty triggers (never View settings/
|
|
97
|
+
Import-Export, which stay full-strength always, confirmed live), so
|
|
98
|
+
the mute belongs on --dashed itself, not gated behind a Filter-only
|
|
99
|
+
modifier. */
|
|
100
|
+
& .${CLASS_PREFIX}${p}-tool--dashed { position: relative; box-shadow: none; background: color-mix(in srgb, ${c.fgPrimary} 2%, transparent); color: color-mix(in srgb, ${c.fgPrimary} 50%, transparent); }
|
|
80
101
|
& .${CLASS_PREFIX}${p}-tool--dashed::before { content: ""; position: absolute; inset: 0; border-radius: inherit; border: 1px dashed color-mix(in srgb, ${c.fgPrimary} 6%, transparent); pointer-events: none; }
|
|
81
102
|
& .${CLASS_PREFIX}${p}-tool--dashed:hover { background: color-mix(in srgb, ${c.fgPrimary} 4%, transparent); }
|
|
82
103
|
& .${CLASS_PREFIX}${p}-toolgroup { position: relative; display: inline-flex; }
|