Skip to content

Add change-block navigation and refine the sidebar handle - #39

Merged
ziqq merged 2 commits into
mainfrom
ziqq/change-block-navigation
Oct 24, 2026
Merged

ziqq merged 2 commits into
mainfrom
ziqq/change-block-navigation

Conversation

@ziqq

@ziqq ziqq commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner

Adds a Changes toolbar with cyclic previous/next block navigation and a scroll-aware counter to file tabs and View full, in Unified and Side-by-side. Navigation reveals virtualized source rows, synchronizes split panes and uses existing tab-position persistence. Shared SVG icon buttons align with the layout switch.

The collapsed sidebar handle stays at the viewport edge instead of following GitHub's Issues navigation. It sits lower with a 17% bottom inset and uses a compact 26px width, 12px text and 14px icons; the obsolete native-navigation observer and positioning calls are removed.

Validation:

  • npm run verify: 292 tests pass; source checks, build and runtime packaging pass.
  • The 19 change-block regressions fail against pre-feature commit 8f6a8da on missing navigation controls and pass with this implementation. They cover both layouts, scrolling, clamped positions, file/PR/MR restoration, viewer isolation and disposal.
  • Manual browser checks on synthetic pages cover diff navigation, toolbar alignment and collapsed handle placement and sizing.

@github-actions github-actions Bot added documentation Improvements or additions to documentation need test's Issue to be tested labels Oct 23, 2026
@ziqq ziqq changed the title Add change-block navigation to diff viewers Add change-block navigation and refine the sidebar handle Oct 23, 2026
@ziqq ziqq self-assigned this Oct 24, 2026
@ziqq
ziqq merged commit 7f3854f into main Oct 24, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation need test's Issue to be tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant