Skip to content

fix(desktop): reduce diff viewer gutter width#861

Merged
Kitenite merged 1 commit into
mainfrom
changes-gutter
Jan 21, 2026
Merged

fix(desktop): reduce diff viewer gutter width#861
Kitenite merged 1 commit into
mainfrom
changes-gutter

Conversation

@Kitenite
Copy link
Copy Markdown
Collaborator

@Kitenite Kitenite commented Jan 21, 2026

Summary

  • Reduces line number column width in Monaco editor by setting lineNumbersMinChars: 3
  • Hides the gutter menu in diff viewer with renderGutterMenu: false to save horizontal space

before:
Screenshot 2026-01-20 at 6 09 28 PM

after:
Screenshot 2026-01-20 at 6 08 40 PM

Test plan

  • Open the changes view in the desktop app
  • Verify the line number gutter is narrower
  • Verify the revert gutter menu is hidden

Summary by CodeRabbit

  • Improvements
    • Enhanced line number display width in the editor for improved visibility
    • Streamlined diff viewer by disabling the gutter context menu for a cleaner interface

✏️ Tip: You can customize this high-level summary in your review settings.

- Add lineNumbersMinChars: 3 to Monaco editor options to reduce line number column width
- Disable renderGutterMenu in diff viewer to hide the revert gutter
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 21, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Two Monaco editor configuration properties are added across separate files: lineNumbersMinChars: 3 is added to the global MonacoProvider options, and renderGutterMenu: false is added to the DiffEditor options in DiffViewer, disabling the gutter context menu.

Changes

Cohort / File(s) Summary
Monaco Editor Configuration
apps/desktop/src/renderer/providers/MonacoProvider/MonacoProvider.tsx, apps/desktop/src/renderer/screens/main/components/WorkspaceView/ChangesContent/components/DiffViewer/DiffViewer.tsx
Added lineNumbersMinChars: 3 to exported MONACO_EDITOR_OPTIONS constant; Added renderGutterMenu: false to DiffEditor options to disable gutter menu in diff view

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 Two tiny tweaks, so neat and clean,
Line numbers arranged, a gutter unseen,
Monaco whispers its settings anew,
Config by config, we polish the view! ✨

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kitenite Kitenite merged commit 8ac5e76 into main Jan 21, 2026
4 of 5 checks passed
@Kitenite Kitenite deleted the changes-gutter branch January 21, 2026 02:10
@github-actions
Copy link
Copy Markdown
Contributor

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

  • ⚠️ Neon database branch
  • ⚠️ Electric Fly.io app

Thank you for your contribution! 🎉

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