Skip to content

feat: center code in diff - #72

Merged
hardbeat920 merged 2 commits into
hardbeat920:mainfrom
tcmarkfeld:feat/move-check-for-updates
Sep 5, 2026
Merged

feat: center code in diff#72
hardbeat920 merged 2 commits into
hardbeat920:mainfrom
tcmarkfeld:feat/move-check-for-updates

Conversation

@tcmarkfeld

@tcmarkfeld tcmarkfeld commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Center code in the diff viewer
Screenshot 2026-09-04 at 5 06 24 PM

Verification

  • npm run build passed
  • tests passed

Warexpor added a commit to Warexpor/monocode that referenced this pull request Sep 4, 2026
…f lines (#21)

Port hardbeat920#72: compact the sidebar update control next to
Settings, and vertically center hunk/gutter/code rows in UnifiedDiffView.

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@hardbeat920

Copy link
Copy Markdown
Owner

Thanks for the contribution @tcmarkfeld !

I like the diff-line centering change and am happy to keep that.
I’d prefer not to move “Check for updates” into the small icon beside Settings.
The project sidebar isn’t cluttered enough yet to justify compressing it, and the icon-only treatment removes useful context such as the current version and update/download status.

Could you revert the sidebar update changes and leave just the diff-centering change in this PR?

Thanks again!

@tcmarkfeld

Copy link
Copy Markdown
Contributor Author

Updated @hardbeat920

@tcmarkfeld tcmarkfeld changed the title feat: consolidate check for updates and center code in diff feat: center code in diff Sep 5, 2026
@hardbeat920
hardbeat920 merged commit 2bc69af into hardbeat920:main Sep 5, 2026
2 checks passed
@hardbeat920

Copy link
Copy Markdown
Owner

@tcmarkfeld thank you. Merging :)

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.

2 participants