style: clean up diff viewer modal and add markdown scroll sync#26
Merged
wesbillman merged 4 commits intomainfrom Feb 11, 2026
Merged
style: clean up diff viewer modal and add markdown scroll sync#26wesbillman merged 4 commits intomainfrom
wesbillman merged 4 commits intomainfrom
Conversation
Remove border outlines from modal, sidebar, pane headers, and toolbars (background colors handle visual separation). Restyle modal as full-width banner with top padding to clear window controls. Replace header bar with floating close button in top-right corner. Remove redundant "N files changed" count (already shown in sidebar). Add Escape key listener via onMount for reliable modal dismissal. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Replace emoji (📄) and text ({ }) with proper Lucide FileText and Code
icons in the markdown preview toggle buttons, consistent with the rest
of the app's icon usage.
Co-Authored-By: Claude Opus 4.6 <[email protected]>
Reduce modal backdrop top padding from 52px to 40px to match the actual top bar height, eliminating the awkward gap that exposed a sliver of the main page. Remove top border-radius so the modal sits flush against the top bar. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Implement proportional scroll sync between the before/after panes in markdown rich preview mode. When one pane scrolls to X% of its content, the other follows to the same proportion. The algorithm is standalone in markdownScrollSync.ts with a feedback-loop guard via rAF. Co-Authored-By: Claude Opus 4.6 <[email protected]>
wesbillman
approved these changes
Feb 11, 2026
loganj
added a commit
that referenced
this pull request
Feb 26, 2026
Refine In Review UI rendering and sidebar behavior
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
Summary
Polishes the diff viewer modal UI and adds scroll synchronization for markdown preview mode.
Changes
Modal styling (DiffModal.svelte)
onMountfor reliable Esc handlingDiff viewer styling (DiffViewer.svelte)
{ }) with LucideFileText/Codeicons for the markdown preview toggleMarkdown scroll sync (new: markdownScrollSync.ts)
syncingguard +requestAnimationFrameto prevent feedback loops$effectthat activates only in markdown preview mode