feat(issues): Use shared markdown component for activity notes#116300
Conversation
Activity notes and the note preview were still using MarkedText with local markdown styles layered on top. Swap both to the core Markdown component and let it own the rendered markdown styling. Also trims some dead note input styling while we are here. Co-Authored-By: Codex GPT-5 <noreply@openai.com>
github-actions
Bot
added
the
Scope: Frontend
📊 Type Coverage Diff✅ No new type safety issues introduced. Coverage: 93.58% |
scttcper
changed the title
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 17b9663. Configure here.
| <Text as="span" size="sm" variant="muted"> | ||
| {t('Markdown supported')} | ||
| </Text> | ||
| </Flex> |
There was a problem hiding this comment.
Flex color prop doesn't set CSS color for icon
Medium Severity
The color="content.secondary" prop on Flex doesn't translate to a CSS color property. The Container/Flex layout component doesn't support a color prop — it only handles layout-related CSS properties like padding, background, border, etc. This prop either fails to compile or gets forwarded as a no-op HTML attribute. The old MarkdownIndicator styled component correctly set color via CSS. The Text child with variant="muted" still gets the right color, but IconMarkdown loses its secondary content color.
Triggered by project rule: Frontend guidelines
Reviewed by Cursor Bugbot for commit 17b9663. Configure here.
Bugbot caught that Flex does not own a color prop, so the markdown icon was not guaranteed to stay muted. Use the icon variant API instead.
2 participants
Activity notes and the note preview were still using MarkedText with local markdown styles layered on top. This swaps both to the core Markdown component and lets it own the rendered markdown styling.
Only real functional change is internal links (missing https://) are no longer allowed, which i think is fine.
before
after