Skip to content

馃悶 Keep PDF navigation accessible in split view - #190

Merged
JFWooten4 merged 1 commit into
mainfrom
split-view-navigation-clearance-20261016
Oct 16, 2026
Merged

JFWooten4 merged 1 commit into
mainfrom
split-view-navigation-clearance-20261016

Conversation

@JFWooten4

Copy link
Copy Markdown
Member

Problem

In a constrained split-view panel, a host-provided control at the lower-right corner can obscure the minimap's controls (including collapse/expand). The footnote scrubber can extend into the same interactive area. The More PDF tools menu also has no height cap, so lower settings such as Show minimap and footnote/navigation options can extend beyond shorter panels.

Changes

  • Reserve 48px of bottom clearance at viewer widths up to 1200px, keeping expanded/collapsed minimap controls clear of the lower-right overlay. Reflect that clearance in the native scrollbar track.
  • In the same constrained widths, inset the footnote scrubber's right-hand control area to prevent a split-view overlay from intercepting scrubber interaction.
  • Constrain the PDF tools dropdown to the viewport's dynamic height and make its content scrollable so the minimap and footnote options remain reachable.
  • Add regression assertions for the split-view clearance and viewport-bounded tools menu.

Verification

  • Confirmed the commit contains changes to exactly three intended files, and the proposed Git tree differs from main.
  • Checked CSS rules against regression assertions for the split-view safe clearance, footnote scrubber and menu scrolling.
  • Not visually verified in a live split-view host; please check expanded/collapsed minimap, the footnote scrubber, scrolling all tools menu settings, and full-window behavior.

Screenshot provided in the originating discussion demonstrates the lower-right overlay.

Reserve a clear bottom-right region for minimap controls and the footnote scrubber at split-pane widths. Keep the PDF tools menu scrollable within the visible viewport and add narrow-layout regression coverage.

Co-authored-by: Codex Web <noreply@openai.com>
@JFWooten4
JFWooten4 merged commit 22c4477 into main Oct 16, 2026
1 check failed
@JFWooten4
JFWooten4 deleted the split-view-navigation-clearance-20261016 branch October 16, 2026 19:36
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