Skip to content

docs(dashboard): TechLabel component spec - #995

Merged
jo-duchan merged 1 commit into
mainfrom
docs/dashboard-tech-label-spec
Oct 18, 2026
Merged

jo-duchan merged 1 commit into
mainfrom
docs/dashboard-tech-label-spec

Conversation

@jo-duchan

@jo-duchan jo-duchan commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Summary

Adds the TechLabel component spec (packages/dashboard/design/components/tech-label.md) for the external Figma designer and links it from DESIGN.md, continuing #977–#994.

TechLabel has no cva; tone (inherit, muted) is declared own: — the color comes from the parent or a call-site class. The spec lists the size and weight a label picks up from its row, and the mono text set by hand without TechLabel; two of those (the Settings bundle ID, the QA session OS version) are drift from DESIGN.md's rule and are marked so. DESIGN.md's Monospace row now says the face is variable and renders at 500 in medium rows.

Checklist

  • Tests written and passing — designSpecMatchesCva covers the new spec (renaming the axis fails it)
  • No any
  • Interface changes land in agent-core first — n/a
  • No sensitive info (tokens, paths, credentials)
  • I verified this change myself
  • AI use disclosed, if any — written with Claude Code

Related .work/ docs

  • Plan: .work/2026-10-17-dashboard-component-spec-plan.md
  • Review: .work/reviews/docs__dashboard-tech-label-spec.md — one adversarial channel, 6 findings, all fixed

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added design guidance for the TechLabel component, including its appearance, tone options, and recommended usage.
    • Updated the typography reference to include medium-weight Monospace text and linked the TechLabel component to its specification.

…r muted color, and the hand-set mono that drifts from it

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
tapflow-docs Ignored Ignored Oct 18, 2026 1:01pm UTC

@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: jo-duchan/tapflow/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 57d1e443-7f9e-40e0-962d-373913e20be1
📥 Commits

Reviewing files that changed from the base of the PR and between d46593a and bbb7130.

📒 Files selected for processing: 2
  • packages/dashboard/DESIGN.md
  • packages/dashboard/design/components/tech-label.md

Included review availability: This review used your included allowance. Your plan provides up to 5 included reviews per hour; 3 remain after this review.

📝 Walkthrough

The dashboard design reference now specifies an additional JetBrains Mono weight and links to a new TechLabel specification.

Changes: Dashboard design documentation

Layer / File(s) Summary
TechLabel specification
packages/dashboard/design/components/tech-label.md, packages/dashboard/DESIGN.md
The new specification describes TechLabel’s visual properties, tone values, usage variations, and guidance. The components table links to the specification.
Monospace weight detail
packages/dashboard/DESIGN.md
The Monospace entry adds JetBrains Mono weight 500 for cases where the parent is medium. It retains weight 400.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to bbb71

This documentation-only change accurately describes TechLabel’s styling and the monospace weight detail. No actionable merge-blocking risk remains.

Pre-merge checks: Passed 5
Check name Status Explanation
Description check Passed The description covers the change, implementation details, checklist, related work documents, testing, verification, and AI disclosure. It also identifies the documentation-only changeset exemption.
Title check Passed The title is concise and accurately identifies the main change: adding the dashboard TechLabel component specification.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@jo-duchan
jo-duchan merged commit 8a49488 into main Oct 18, 2026
16 checks passed
@jo-duchan
jo-duchan deleted the docs/dashboard-tech-label-spec branch October 18, 2026 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant