Skip to content

feat(design-systems): add riso bundled design system with localized summaries - #8557

Open
palprateek wants to merge 1 commit into
nexu-io:mainfrom
palprateek:split-design-system-riso
Open

palprateek wants to merge 1 commit into
nexu-io:mainfrom
palprateek:split-design-system-riso

Conversation

@palprateek

@palprateek palprateek commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Split from #8135 per product sign-off β€” riso design system (5 of 6). The source PR bundled 10 skills, 2 design systems, and a method fold-in; each bundled system needs its own product decision. This PR is independently reviewable and landable.

Why

The bundled catalogue had no warm print-shop aesthetic β€” playful, high-contrast, two-ink design work defaulted to neutral SaaS chrome. Riso adds a risograph system (warm paper, rose interaction accent, federal-blue structure) so joyful/creative artifacts have a native starting point, with contrast repaired to the AA contract before shipping (the first version failed at 3.60:1).

What users will see

A new Riso card in the Design Systems catalogue and picker: playful two-color risograph aesthetic β€” warm paper canvas (#fdf6ec), deep rose interaction accent (#c2187b, white text 5.66:1), federal-blue structure (#2c40a7), Space Grotesk + Overpass Mono, offset print-shadows and pink-glow elevation. All 8 documented fixture text pairs meet WCAG AA (4.5:1), pinned by a regression test. Picker summaries render localized in all 17 locale bundles (e.g. JA), falling back to English only for genuinely untranslated systems.

Example: picking Riso for a landing page yields the warm-paper + two-ink system (pink CTAs/links, blue headings, grain texture) instead of the default system styling.

Surface area

  • Extension point β€” new bundled system under design-systems/riso/
  • i18n keys β€” riso picker summary added to all 17 designSystemSummaries locale maps (pure additions; the source PR had replaced runwayml/tesla in DE/FR/RU β€” neighbors restored here, so this also fixes that regression)
  • UI
  • Keyboard shortcut
  • CLI / env var
  • API / contract
  • New top-level dependency
  • Default behavior change
  • None

apps/web/src/i18n/content.ts also gains the DIRECT_LOCALE_DESIGN_SYSTEM_IDS export (all-locales id view for regression tests; the byte-identical hunk is in the terracotta split too and 3-way-merges clean).

Validation

@lefarcen

lefarcen commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Thanks @palprateek β€” the Riso split is clearly scoped, including the locale coverage and dedicated regression coverage. I've routed it for maintainer review.

πŸ’‘ To drive this PR to merge hands-free, paste this to your AI coding agent (Claude Code / Codex / opencode / Cursor …):
Take over nexu-io/open-design#8557 until it merges β€” read https://raw.githubusercontent.com/nexu-io/looper/main/skills/pr-takeover/SKILL.md and follow it.

@lefarcen
lefarcen requested a review from mrcfps October 1, 2026 17:03
@lefarcen lefarcen added needs-product-review Feature PR awaiting product sign-off before merge (see roadmap) needs-design-review Awaiting design review (external UI change); cleared by the Odcrew App applying design-approved needs-validation Runtime change detected; needs human or /explore agent validation. labels Oct 1, 2026
@lefarcen

lefarcen commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

πŸ§ͺ This PR has changes that need a manual QA pass before merge β€” please hold off self-merging for now; we’ll loop QA in once it’s merge-ready and design/product have signed off, where applicable.

@lefarcen lefarcen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The split context and user-facing Riso description make the intended catalogue addition clear. Could you add a brief Why, a checked Surface area section, and a Validation section so review and release verification have the full context? Related: #8135 is the source split and #8558 is its Terracotta sibling.

@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Visual regression review

Head: ad2b8e3 Β· Base: 53231d4

0 changed Β· 53 unchanged Β· 0 new without baseline Β· 0 failed

Unchanged cases
Case Main PR Diff
visual-avatar-local-agent-list
0 px (0.00%)
main pr diff
visual-avatar-local-agent-list-panel
0 px (0.00%)
main pr diff
visual-avatar-menu
0 px (0.00%)
main pr diff
visual-avatar-menu-panel
0 px (0.00%)
main pr diff
visual-avatar-open-design-model-picker
0 px (0.00%)
main pr diff
visual-critical-settings
0 px (0.00%)
main pr diff
visual-critical-workspace
0 px (0.00%)
main pr diff
visual-critical-workspace-preview
0 px (0.00%)
main pr diff
visual-deepseek-unpaid-campaign-600
0 px (0.00%)
main pr diff
visual-deepseek-unpaid-campaign-short-height
0 px (0.00%)
main pr diff
visual-design-system-detail
0 px (0.00%)
main pr diff
visual-design-systems
0 px (0.00%)
main pr diff
visual-home
0 px (0.00%)
main pr diff
visual-home-catalog
0 px (0.00%)
main pr diff
visual-home-context-picker
0 px (0.00%)
main pr diff
visual-home-context-picker-popover
0 px (0.00%)
main pr diff
visual-home-plugin-filter
0 px (0.00%)
main pr diff
visual-home-plugin-use-staged
0 px (0.00%)
main pr diff
visual-home-plugin-use-with-query
0 px (0.00%)
main pr diff
visual-home-staged-attachment
0 px (0.00%)
main pr diff

Visual diff is advisory only and does not block merging.

@palprateek

Copy link
Copy Markdown
Contributor Author

@mrcfps the riso design-system split is ready for review at head ad2b8e3 β€” CI fully green, no threads. @lefarcen's body asks are done (Why / Surface area / Validation now in the PR body, including the runwayml/tesla neighbor-restore note). I lack permission to formally re-request, so treating this as the review nudge. QA/product/design holds respected β€” no self-merge.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-design-review Awaiting design review (external UI change); cleared by the Odcrew App applying design-approved needs-product-review Feature PR awaiting product sign-off before merge (see roadmap) needs-validation Runtime change detected; needs human or /explore agent validation. risk/medium Medium risk: regular code changes size/L PR changes 300-700 lines type/feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants