Claude/v2 spacing workbench sync 38b80s - #7
Merged
Conversation
…tory - Promote every workbench slider to a named token with workbench defaults: --v-gap-section (1.5x) and --v-gap-tight (0.25x) join the spacing bundle; pad becomes 1.25x; input-height 2rem; radius anchors 2.5rem/0.375rem; --o-prose__heading-gap, --o-hgroup__leading, --v-list-rhythm defaults; new --v-card-bg, --v-input-bg, --v-backdrop, --v-border-color-strong - Spec o-bar fully from workbench: one symmetric --o-bar__pad, three mode formulas (text/contains/input), sticky glass bg, one-row shrink contract, corner occupancy; top-level section flow with heading-led exception - Scope text-box-trim to o-prose/o-stack text children via selector list; document the flex-container caveat; controls center by height math only - Add c-table: --c-table__row-height defaults to --v-input-height but is decoupled; cell metrics use the control recipe, trim only as refinement - Allow one nested o-box level via --o-box__radius-inner (deeper nesting bottoms out at the floor); box-in-box supported, card-in-card discouraged - De-emphasize mixins: composition happens in markup (no o-card, no class mixing); formalize unlayered p- project prefix with discipline rules - Author the demo site in Solid JSX + SSG from day one (render-only wrappers grow with the CSS; zero client JS on CSS-reference pages) - Replace config modal + per-token slider islands with a three-tier customization story: site-wide theme workbench drawer (the page is the preview), scoped --o-* knob demos on the Objects page, mode toggles on component demos - Make full-bleed a per-frame opt-in (data-v-bleed); split t-align-* (box) from t-text-* (text alignment); fix --v-font-ratio-max default to 1.2 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DGKBZeR13TPtYn2Lp7XeV
…ed object modifier - Remove o-text-box everywhere: text-box-trim removed the compensated-padding reason for a separate text box, so one padded box serves both content kinds. Prose in a box is o-box + o-prose (box owns perimeter/radius/curvature clearance; prose owns trim, rhythm, inline insets). Curvature clearance moves onto o-box itself; pre/blockquote/td stay base.css-padded text boxes. prose.css moves up to step 2 of the Phase 2 build order accordingly. - Replace data-v-bleed with the --bleed frame modifier (o-container--bleed / o-dialog--bleed), following the o-grid--fit object-modifier precedent: full-bleed is an authoring-time structural mode, so a class — data-v-* is reserved for runtime state, and a custom property can't gate @container conditions (style() queries aren't dependable at the browser floor). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DGKBZeR13TPtYn2Lp7XeV
v- classes may now flip documented modes of the o-/c- classes beneath them, not just set custom properties — with two guardrails: variant rules target library classes in their subtree only (a variant has no look of its own), and every mode a variant flips is documented on the class it flips. Rename the full-bleed opt-in from the o-container--bleed / o-dialog--bleed modifiers to a single v-bleed variant (variants/bleed.css) that works on any frame. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DGKBZeR13TPtYn2Lp7XeV
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.