Skip to content

feat(chrome): put Settings beside update check and center unified-diff lines - #21

Merged
Warexpor merged 1 commit into
mainfrom
cursor/sidebar-update-diff-chrome-79f9
Sep 4, 2026
Merged

feat(chrome): put Settings beside update check and center unified-diff lines#21
Warexpor merged 1 commit into
mainfrom
cursor/sidebar-update-diff-chrome-79f9

Conversation

@Warexpor

@Warexpor Warexpor commented Sep 4, 2026

Copy link
Copy Markdown
Owner

What changed

Port of hardbeat920/monocode#72 onto this fork’s current chrome:

  1. Sidebar footer — Settings sits on the same row as a compact size-8 update-check icon in SidebarUpdateFooter. Label + app version live on title / aria-label. The separate Settings RailAction blocks were removed from ProjectRail.tsx and Sidebar.tsx.
  2. Unified diff — Hunk headers, gutter numbers, and code lines in UnifiedDiffView.tsx use flex items-center + leading-none instead of explicit lineHeight.

Why

Match upstream hardbeat920#72 without redesigning MonoCode chrome (still RailAction, UpdateRailCard, and accent states). Checking / available / download affordances stay on the update button via title.

Checklist

  • I ran npm run check:web (vitest + tsc; 117 files / 1208 tests)
  • npx vite build succeeded
  • This PR is small and focused
  • I did not mix unrelated changes
  • Browser-smoked React chrome in Vite (not the Tauri shell)

Verification

Browser (Vite, no Tauri) — temporary harness mounted SidebarUpdateFooter + UnifiedDiffView (not committed):

  • Settings RailAction and update icon sit on one row (Settings ~203×32, update 32×32, same top).
  • Clicking Settings still fires onOpenSettings.
  • After the browser updater catch path: title / aria-label = Check for updates · v0.0.0 (idle; getVersion falls back, check() is not configured).
  • Unified diff code/gutter rows use flex + align-items: center and leading-none.

The full index.html App boot still depends on Tauri (loadBootWorkspace / invoke); that shell was not exercised.

Needs Tauri (not claimed here): live check() probe, checking/available/download icon states against a real endpoint, install/relaunch, native dialogs, Windows/DWM.

Refs: hardbeat920#72

Open in Web Open in Cursor 

…f lines

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

Co-authored-by: Warexpor <warexpor@gmail.com>
@Warexpor
Warexpor marked this pull request as ready for review September 4, 2026 21:28
@Warexpor
Warexpor merged commit fa27a19 into main Sep 4, 2026
4 checks passed
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