Skip to content

Refactor note rendering and add class archive support#56

Merged
AliBenrami merged 2 commits into
developfrom
Ali/add-mermaid-note-component
Jun 11, 2026
Merged

Refactor note rendering and add class archive support#56
AliBenrami merged 2 commits into
developfrom
Ali/add-mermaid-note-component

Conversation

@AliBenrami

Copy link
Copy Markdown
Collaborator

Summary

  • Split the note editor into smaller block-focused modules for code, math, Mermaid, clipboard, selection, and rendering helpers.
  • Fix note rendering so inline Markdown markers like bold, links, and code render correctly in generated notes instead of leaking raw syntax.
  • Improve block selection so text selection inside note content is handled more reliably and no longer fights block drag behavior as often.
  • Add a real class archive action, hide archived classes from the classes list, and surface class editing in the class detail sidebar.
  • Tune note generation to produce fewer, denser topic notes with more content per note.

Testing

  • Added and updated unit coverage for note parsing, records normalization, generation density, selection geometry, and note editor behavior.
  • Verified the note-focused test suite and TypeScript checks pass locally.
  • Ran lint; remaining warnings are limited to pre-existing unused imports in quiz and flashcard clients.

@AliBenrami AliBenrami requested a review from hamizfaraz as a code owner June 8, 2026 14:40
@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
task-master Ready Ready Preview, Comment Jun 9, 2026 3:50am

@AliBenrami AliBenrami merged commit f56974d into develop Jun 11, 2026
3 checks passed
@AliBenrami AliBenrami deleted the Ali/add-mermaid-note-component branch June 11, 2026 02:53
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