chore(skills): re-vendor devup-ui for the reset scope and token alpha - #72
Merged
Merged
Conversation
devup-ui@c7295916 answers two things readers had been working out the hard way, and devup-mcp ships that document inside the binary, so `devup_skills install` writes it on machines with no network - the machines least able to work either answer out for themselves. `@devup-ui/reset-css` is a normalize, not a preflight. For form controls it sets `margin: 0` and `-webkit-appearance: button`, and the second rule *preserves* the native control appearance rather than removing it, so button `padding`, `border` and `appearance` are deliberately untouched. That stays invisible until a button is sized in design units: `box-sizing: border-box` is global, so an explicit `8px` width cannot shrink below the `6 + 6 + 2 + 2 = 16px` that Chrome's default `padding: 1px 6px` and `border: 2px outset` already occupy, and an 8x8 button from a design renders 16x8. That is the reset behaving as specified, not a defect, and the document now says so with the call-site fix. The other is alpha. A token is a CSS custom property, which that document's `$token Scope` section already establishes, so a token at partial opacity is `color-mix()` over `var(--token)` - no alpha token to define, no copy of the colour to keep in sync. Without it a screen hardcoded `#F7F3EC66` beside a `$bg` of the same value, which is not merely redundant but wrong under theming: `$bg` follows the active theme and the literal does not. Produced by `node scripts/refresh-skills.mjs`; `--check` reports no remaining drift and the manifest integrity test passes.
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
1 participant
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.
Follows dev-five-git/devup-ui#671. Produced by
node scripts/refresh-skills.mjs;--checknow reports no remaining drift and the manifest integrity test passes.devup-mcp vendors this document into the binary, so
devup_skills installwrites it on machines with no network — the machines least able to work either answer out for themselves.What the document now answers
The reset leaves buttons alone.
@devup-ui/reset-cssis a normalize, not a preflight: for form controls it setsmargin: 0and-webkit-appearance: button, and that second rule preserves the native control appearance rather than removing it.padding,borderandappearanceare deliberately untouched.That is invisible until a button is sized in design units. With
box-sizing: border-boxglobal, an explicit8pxwidth cannot shrink below the6 + 6 + 2 + 2 = 16pxthat Chrome's defaultpadding: 1px 6pxandborder: 2px outsetalready occupy — an 8x8 button from a design renders 16x8. The reset behaving as specified, not a defect.A token at 40% already has an answer. A token is a CSS custom property, which that document's
\ Scopesection already establishes, so partial opacity iscolor-mix()overvar(--token)— no alpha token to define, no copy of the colour to keep in sync. Without it a screen hardcoded#F7F3EC66beside a\of the same value, which is wrong under theming and not merely redundant:\follows the active theme and the literal does not.Diff
Vendored copy
601dd15c -> c7295916(21,253 -> 23,101 bytes), with the manifest's commit, digest, byte count and source URL resealed to match.