improvement(mothership): align markdown blockquote, img, em, del with design tokens#4566
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Adds explicit workspace chat markdown overrides for Reviewed by Cursor Bugbot for commit 01ca881. Configure here. |
Greptile SummaryThis PR replaces hardcoded Tailwind gray/border values in markdown blockquote renderers across four files with design tokens (
Confidence Score: 5/5Safe to merge — changes are purely presentational class replacements with no logic modifications. All four files receive straightforward string substitutions swapping hardcoded gray palette values for design tokens. The Tailwind variant scoping bug flagged in earlier review rounds is correctly fixed in this head commit. No data flows, API calls, or component props are altered, so the risk surface is limited to visual regressions only. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Markdown Input] --> B{Renderer Context}
B -->|Public Chat Route| C[markdown-renderer.tsx]
B -->|File Viewer| D[preview-panel.tsx]
B -->|Mothership Home Chat| E[chat-content.tsx]
B -->|Note Block Canvas| F[note-block.tsx]
C --> G[blockquote: --divider / --text-primary]
D --> H[CalloutBlock fallback: --divider / --text-primary]
D --> I[STATIC_MARKDOWN_COMPONENTS: --divider / --text-primary]
E --> J[blockquote: --divider / --text-primary]
E --> K[em: --text-primary italic]
E --> L[del: --text-tertiary line-through]
E --> M[img: lazy-loaded, border --divider]
F --> N[blockquote: --divider / --text-primary]
F --> O[em: --text-tertiary unchanged]
Reviews (2): Last reviewed commit: "fix(mothership): correctly scope blockqu..." | Re-trigger Greptile |
… first/last child
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 01ca881. Configure here.
Summary
blockquote,img,em, anddeloverrides to the Mothership markdown renderer so they use design tokens instead of prose defaultsblockquotestyling across note-block, file viewer (callout fallback + main), and chat route toborder-[var(--divider)] border-l-2 pl-4 text-[var(--text-primary)](no italic, no hardcoded grays)Type of Change
Testing
Tested manually in Mothership chat.
Checklist