hugo/Gotcha
1
0
Fork 0
Code Issues 13 Pull Requests Actions Packages Projects Releases Wiki Activity

Diffs are word wrapped #31

New Issue
Closed
opened 2026-07-31 14:36:48 +00:00 by hugo · 1 comment
hugo commented 2026-07-31 14:36:48 +00:00
Owner
Copy Link

Diffs must not be word wrapped. Instead they need to be presented like source is, with a scrollable area, using the exact same UI elements. Essentially diffs are just another form of source and must use the same presentation.

Diffs must not be word wrapped. Instead they need to be presented like source is, with a scrollable area, using the exact same UI elements. Essentially diffs are just another form of source and must use the same presentation.
hugo added this to the first feature complete release milestone 2026-07-31 14:36:48 +00:00
hugo added the bug label 2026-07-31 14:36:48 +00:00
hugo commented 2026-07-31 15:15:01 +00:00
Author
Owner
Copy Link

Implemented in 5b29e18.

  • Replaced the wrapping diff text view with the same native CodeScrollView now used by repository source files.
  • Commit and pull-request diffs preserve line numbers, monospaced text, selection, refresh, and addition/removal/hunk colors while clipping long lines and supporting horizontal and vertical scrolling.
  • Consolidated source layout into the shared view, removed the duplicate manual layout path, respected the navigation safe area, and updated TESTING.md with the no-wrap regression checks.

Verified in isolation with the full required gate suite (cargo fmt, warning-free workspace check, Clippy with warnings denied, and all workspace tests), a signed arm64 iPhone 17 Pro simulator build, and manual simulator checks of a long commit diff plus the shared repository source view. The diff remained unwrapped and pannable with colors and line numbers intact; source highlighting and scrolling remained correct.

Implemented in 5b29e18. - Replaced the wrapping diff text view with the same native `CodeScrollView` now used by repository source files. - Commit and pull-request diffs preserve line numbers, monospaced text, selection, refresh, and addition/removal/hunk colors while clipping long lines and supporting horizontal and vertical scrolling. - Consolidated source layout into the shared view, removed the duplicate manual layout path, respected the navigation safe area, and updated `TESTING.md` with the no-wrap regression checks. Verified in isolation with the full required gate suite (`cargo fmt`, warning-free workspace check, Clippy with warnings denied, and all workspace tests), a signed arm64 iPhone 17 Pro simulator build, and manual simulator checks of a long commit diff plus the shared repository source view. The diff remained unwrapped and pannable with colors and line numbers intact; source highlighting and scrolling remained correct.
hugo closed this issue 2026-07-31 15:15:10 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 critical
critical issues that take precendence over others of the same grouping
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 idea
an idea for a future milestone
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label bug
Milestone
No items
No Milestone first feature complete release
Projects
Clear projects
No project
Assignees
Clear assignees
hugo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: hugo/Gotcha#31
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?