Feature: add interactive Mermaid diagram rendering with pan-zoom #261
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…m support - Add MermaidRenderer component with fullscreen dialog viewer - Integrate @panzoom/panzoom for smooth pan and zoom interactions - Replace auto-rendering with on-demand code block buttons (copy + render) - Add intelligent height fitting and vertical centering for optimal viewing - Support copy feedback with visual indicators - Add accessibility compliance with proper DialogTitle - Maintain performance with caching and intersection observer optimization
kmk142789
approved these changes
Sep 22, 2025
|
Oh this is looks cool. I see its been a while since you've submitted this. Do you still want to work on this? |
Yes, absolutely! I'm still interested in working on this. |
Sallvainian
added a commit
to Sallvainian/opcode
that referenced
this pull request
Oct 16, 2025
- Add interactive Mermaid diagram rendering with pan-zoom support - Implement MermaidRenderer component with fullscreen dialog viewer - Add global SVG caching strategy with timestamp expiry - Integrate MermaidCodeBlock into StreamMessage for code blocks - Add zoom controls and copy-to-clipboard functionality - Install dependencies: mermaid, @panzoom/panzoom, @types/mermaid - Resolve cherry-pick conflicts from PR winfunc#261 - Fix duplicate code blocks from automated conflict resolution - Clean up unused imports and variables 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Sallvainian
added a commit
to Sallvainian/opcode
that referenced
this pull request
Oct 16, 2025
Added logging to confirm all integrated features work at runtime: - PR winfunc#261 (Mermaid): Track cache hits and render success - PR winfunc#222 (MCP Async): Track sidecar vs regular execution - PR winfunc#346 (NVM PATH): Track Node.js PATH additions All logs prefixed with PR numbers for easy identification. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Old：
New：
Need Help：
Currently unable to render Mermaid diagrams directly within the MessageList due to virtual scrolling implementation. The virtual scrolling causes components to repeatedly unmount/mount during scroll operations, resulting in constant flickering and re-rendering of Mermaid SVGs. This approach with on-demand dialog rendering solves the performance issue, but if anyone has experience with stable Mermaid rendering in virtualized lists, feedback would be appreciated.
Issue Resolution
This change potentially provides a temporary workaround solution for Issue #249, which likely relates to Mermaid diagram rendering problems in the message interface