[Design] Redesign decisions and resolved comments
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 38/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- bun, typescript
Research direction
Start with /design-audit#surfaces and inspect the decisions rail, questionnaire cards, comment thread cards, SidecarCard, and provenance in packages/editor. Review the listed verification commands and inventory every required layout and relationship state; done means annotated before/after evidence, shared anatomy decisions, browser evidence, deferred states, and Maggie's approval without changing record authority or workflows.
Written by the indexing model from the issue text.
Description
Base branch: main
Branch: maggie/redesign-decisions-and-comments
Depends on: None
Goal
Give decisions and resolved comments a coherent, high-quality sidecar design while preserving the distinct durable record states.
Context From Planning
- The current decisions and resolved-comment specimens in
/design-audit#surfacesboth need a proper design pass. - They share sidecar-card foundations and provenance, but questions, accepted comments, resolved history, deliberately empty links, and orphaned links must remain semantically distinct.
- This is a human-reviewed design task rather than a mechanical restyle.
Current Behavior
Questionnaires and comment threads share SidecarCard, but information hierarchy, provenance, actions, settled treatment, history disclosure, and empty/orphaned records remain visually weak. Existing behavior carefully distinguishes pending, linked, deliberately empty, and orphaned relationships.
Desired Behavior
Design a small sidecar record system that makes open work, settled decisions, provenance, related prose, and exceptional relationship states immediately legible without fragmenting the product into unrelated card styles.
Acceptance Criteria
- The design audit covers outstanding questionnaire, answered questionnaire, open comment, accepted comment, resolved history, deliberately empty relation, orphaned relation, focused/related-prose, disabled, and narrow layouts.
- Questions and comments share card anatomy/tokens where their semantics match and remain distinct where actions differ.
- Provenance, primary content, contextual quote, related-prose state, and actions have a clear reading order.
- Pending, linked, deliberately empty, and orphaned states remain visibly and programmatically distinct.
- Existing accept/dismiss confirmation, frozen accepted threads, resolved-history disclosure, focus handoff, related-prose highlight/pin, and record authority remain unchanged.
- Maggie approves the audit specimens before production migration.
Verification Commands
bun run typesbun run cibun test packages/editorbun run e2e
Primary Surfaces
/design-audit#surfaces- Decisions rail and questionnaire cards in
packages/editor - Comment thread cards,
SidecarCard, and provenance
Expected PR Checks
- Repo required checks
Report Requirements
- State inventory and annotated before/after screenshots
- Shared anatomy/token decisions
- Browser evidence for focus, disclosure, and related-prose behavior
- Any deliberately deferred states
Implementation Notes
- Records own decisions; rendered document projections are not authority.
- Preserve the four relationship states: pending, linked, deliberately empty, and orphaned.
- Empty means intentional absence; orphaned means the former target cannot be recovered safely.
- Keep one shared foundation without forcing question-owned controls into comment-owned action slots.
Stop Conditions
- A visual treatment obscures the difference between empty and orphaned relationships.
- The design requires changing record authority, anchoring, or questionnaire/comment workflows.
- Dominant language
- TypeScript
- Stars
- 351
- Forks
- 20
- Avg merge
- 14h 7m
- Merged PRs (30d)
- 34
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from githubnext/chopin
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
githubnext/chopin#154 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
githubnext/chopin#149 ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
githubnext/chopin#165 ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
githubnext/chopin#164 ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
githubnext/chopin#162 ·
All issues in githubnext/chopin
Similar issues
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Crush Open
Difficulty 1/5 Under an hour Newbie friendliness 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Open
Difficulty 1/5 Under an hour Newbie friendliness 90/100
ElementsProject/cln-application#167 · 1 comment · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Quantco/pnpm-licenses#17 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100