Skip to content

Replace custom diff parser with diff2html#166

Open
jrchatruc wants to merge 4 commits intomainfrom
feat/diff2html-viewer
Open

Replace custom diff parser with diff2html#166
jrchatruc wants to merge 4 commits intomainfrom
feat/diff2html-viewer

Conversation

@jrchatruc
Copy link
Copy Markdown
Collaborator

Summary

  • Replace the minimal custom diff parser with the already-installed diff2html library
  • Adds: file list summary, line numbers, word-level change matching, auto dark/light theme
  • Tailwind overrides adapt diff2html default styles to match the dashboard theme

Part of #158

Test plan

  • Verify diff tab renders correctly for tasks with changes
  • Verify file list appears at the top
  • Verify additions/deletions are color-coded correctly in both light and dark mode
  • Verify "No diff available yet" still shows for tasks without changes

Use the already-installed diff2html library instead of the minimal
custom parser. Adds file list, line numbers, word-level matching,
and auto dark/light color scheme. Tailwind overrides adapt the
default styles to the dashboard theme.
The colorScheme option requires the ColorSchemeType enum value,
not a plain string.
@jrchatruc jrchatruc marked this pull request as ready for review April 14, 2026 21:28
Line numbers used position: absolute which broke when the diff
was inside a scrollable container. Switch to position: sticky
so they stay aligned with their rows.
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