dries/ocman
1
0
Fork 1
Code Issues 38 Pull requests 4 Projects Releases 122 Packages Wiki Activity Actions 1

feat: add fullscreen diff browser #518

Merged
dries merged 2 commits from feat/fullscreen-diff-modal into main 2026-08-20 09:25:27 +02:00
Conversation 1 Commits 2 Files changed 8 +488 -7
dries commented 2026-08-19 20:31:40 +02:00
Owner
Copy link

Summary

  • add fullscreen controls to Session Changes and Working Tree headers
  • show changed files and the selected diff in a 95% two-column modal
  • reuse existing diff renderers and cover modal selection and close behavior

Testing

  • make lint
  • make test
## Summary - add fullscreen controls to Session Changes and Working Tree headers - show changed files and the selected diff in a 95% two-column modal - reuse existing diff renderers and cover modal selection and close behavior ## Testing - make lint - make test
feat: add fullscreen diff browser
All checks were successful
CI / Frontend (pull_request) Successful in 8m41s
Details
CI / Playwright E2E (pull_request) Successful in 9m34s
Details
CI / Backend (pull_request) Successful in 12m33s
Details
CI / Build (pull_request) Successful in 4m28s
Details
CI / Coverage Results (pull_request) Successful in 21s
Details
CI / Semantic Tag (pull_request) Successful in 6s
Details
2ac09ae73f
forgejo-actions commented 2026-08-19 20:54:18 +02:00
Copy link

Coverage ratchet: ✅ pass

Suite Baseline This PR Δ
frontend 71.22% 71.26% +0.04 ✅
go 81.30% 81.50% +0.20 ✅

Tolerance: -0.1%. Baseline stored on gh-pages.

<!-- coverage-ratchet --> ### Coverage ratchet: ✅ pass | Suite | Baseline | This PR | Δ | | |---|---|---|---|---| | frontend | 71.22% | 71.26% | +0.04 | ✅ | | go | 81.30% | 81.50% | +0.20 | ✅ | _Tolerance: -0.1%. Baseline stored on `gh-pages`._
refactor: dedupe fullscreen diff wiring into useFullscreenDiff hook
All checks were successful
CI / Frontend (pull_request) Successful in 8m1s
Details
CI / Playwright E2E (pull_request) Successful in 9m14s
Details
CI / Backend (pull_request) Successful in 10m30s
Details
CI / Coverage Results (pull_request) Successful in 18s
Details
CI / Build (pull_request) Successful in 5m29s
Details
CI / Semantic Tag (pull_request) Successful in 6s
Details
ed452597d6
dries merged commit 9bd4487fdd into main 2026-08-20 09:25:27 +02:00
dries deleted branch feat/fullscreen-diff-modal 2026-08-20 09:25:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend

Go backend code

  
bug

Something is broken or incorrect

  
chore

Maintenance, tooling, or cleanup

  
duplication

Code duplication to be addressed

  
effort:complex

Significant effort or risk

  
effort:medium

Moderate effort required

  
effort:trivial

Quick fix, low complexity

  
enhancement

New feature or capability

  
follow-up

Follow-up from a PR review or shipped change

  
frontend

TypeScript / React frontend code

  
fullstack

Touches both backend and frontend

  
priority:high

Should be addressed soon

  
ready-for-agent

Agent-grabbable ticket, fully specced

  
refactor

Code restructuring without behaviour change

  
security

Security related

No labels
backend
bug
chore
duplication
effort:complex
effort:medium
effort:trivial
enhancement
follow-up
frontend
fullstack
priority:high
ready-for-agent
refactor
security
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dries/ocman!518
No description provided.