fix(next): clear the webviews type-scale, text-focus and vp fmt base reds

Merged
TE
17d ago
+217-57

Summary

Three webviews checks fail at the feat-cmux-next tip (4bc289e7890) and so on every PR's react-apps-check:

CheckBroken byFix
bun run check:type-scale: syncMotion.gallery.css: 3 raw font sizes, baseline 0c611de179d37 (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 = .none60807d3143d0 (cookie import consent panel)label.focusRingType = .none
vp fmt --check: src/pages/settings/icons.tsx91424a7d4d0e (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


View with [code]smith Autofix with [code]smith
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.css replaces raw font sizes with type-scale tokens; subagents.css and the markdown styles use --cmux-hairline and --elevation-2 for the elevation checks.
  • CookieImportConsentPanel.swift sets focusRingType = .none on its selectable label and draws the requester link with Palette.accent, re-resolved in the theme hook so the tint follows theme changes.
  • syncMotion.ts, useSyncListMotion.ts, settings/icons.tsx, the markdown WidthField, and the #20297 update-card files clear the lint and vp fmt --check reds.
  • Markdown paper colors are marked theme-fixed; the theme-colors test now skips prototypes/. The code editor's keydown listener carries a ui-allow marker.
  • agent-pane-older-rows-fade.test.ts warms the in-process Vite server before the first load to pass check: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.rulers renders 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 TableView so 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 reset scrollLeft on the first wheel step.