fix(next): clear the webviews type-scale, text-focus and vp fmt base reds
Merged
TE
17d ago+217+217+012345678901234567890123456789-57-57-01234567890123456789
Summary
Three webviews checks fail at the feat-cmux-next tip (4bc289e7890) and so on every PR's react-apps-check:
| Check | Broken by | Fix |
|---|---|---|
bun run check:type-scale: syncMotion.gallery.css: 3 raw font sizes, baseline 0 | c611de179d37 (sync motion gallery) | the three sizes use type-scale tokens: 13px -> --text-body, 22px -> --text-heading (20), 16px -> --text-title (15). Gallery-only frame. |
bun run check:text-focus: CookieImportConsentPanel.swift:149: a selectable AppKit label without focusRingType = .none | 60807d3143d0 (cookie import consent panel) | label.focusRingType = .none |
vp fmt --check: src/pages/settings/icons.tsx | 91424a7d4d0e (Settings > Code Editor) | formatter output only |
The failing tests are those checks themselves, red at the base.
Testing (Huge Blue, this head)
check:type-scale: ok (384 left in the baseline). Base: red.check:text-focus: no focus ring on a text field. Base: red.vp fmt --check: clean. Base: icons.tsx flagged.
Changelog: none
š¤ Generated with Claude Code
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by cubic
Clears the webviews check reds at the feat-cmux-next tip so react-apps-check is green at the base.
syncMotion.gallery.cssreplaces raw font sizes with type-scale tokens;subagents.cssand the markdown styles use--cmux-hairlineand--elevation-2for the elevation checks.CookieImportConsentPanel.swiftsetsfocusRingType = .noneon its selectable label and draws the requester link withPalette.accent, re-resolved in the theme hook so the tint follows theme changes.syncMotion.ts,useSyncListMotion.ts,settings/icons.tsx, the markdownWidthField, and the #20297 update-card files clear the lint andvp fmt --checkreds.- Markdown paper colors are marked
theme-fixed; the theme-colors test now skipsprototypes/. The code editor's keydown listener carries aui-allowmarker. agent-pane-older-rows-fade.test.tswarms the in-process Vite server before the first load to passcheck:browser-lanes.- The settings export regenerates the schema checksums and
scopes.json.
Follow-up test reds revealed once the lint red cleared
- Markdown Viewer stays in General as one row; Agent Input (
browser.showAgentCursor) gets a home in Browser after Bookmarks. editor.rulersrenders a new number token field: Return or comma adds a number in range, invalid values show an alert.- The tooltip contrast test draws with the host's per-theme tint instead of the pages' 60% fallback.
- Composer tests turn the plain footer off where they check chip look; the harness spinner gets a Static rule.
- The markdown editor installs
TableViewso wide tables scroll in.tableWrapper; its view ignores attribute changes on the wrapper, which the scroller's marker would otherwise read as an edit and resetscrollLefton the first wheel step.