Tags: ClickHouse/click-ui
Tags
fix(Icon): enlarge check glyph to fill its viewBox The `check` glyph was small and offset within its 24x24 viewBox, so it rendered noticeably smaller than other icons. Scale the path up ~1.25x about the viewBox center; stroke weight stays at 2. Render the Checkbox's check at 0.8rem (= 1rem / 1.25) so the checkmark inside the box keeps its current size; the indeterminate (minus) icon is unaffected. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
fix(Container): stop nested Container inheriting layout custom proper… …ties The CSS Modules migration forwards optional layout props (fillHeight, grow, shrink, minHeight, maxHeight, overflow) to the DOM as CSS custom properties, emitted inline only when the prop is set. Custom properties inherit by default, so a nested Container that omits those props inherited its ancestor's values — e.g. a layout root with `fillHeight` forced `height: 100%` onto every descendant Container (observed as the control-plane alert banner stretching to fill the viewport, with knock-on spacing breakage throughout the page). Reset these custom properties to `initial` on the base `.container` rule so a declaration on the element wins over inheritance, and add `var()` fallbacks matching the styled-components defaults. Verified in a browser: with the bug a child Container inherited flex-grow:1/height:198px/overflow:hidden; after the fix it hugs its content (flex-grow:0, height:34px, overflow:visible). Adds a NestedInheritance story and a computed-style regression test; all 30 existing Container visual snapshots are unchanged. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
fix(Icon): enlarge check glyph to fill its viewBox The `check` glyph was small and offset within its 24x24 viewBox, so it rendered noticeably smaller than other icons. Scale the path up ~1.25x about the viewBox center; stroke weight stays at 2. Render the Checkbox's check at 0.8rem (= 1rem / 1.25) so the checkmark inside the box keeps its current size; the indeterminate (minus) icon is unaffected. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
PreviousNext