Skip to content

fix(ui): restore inset Help scrollbar and scroll-hint title#460

Merged
domcyrus merged 1 commit into
mainfrom
fix/help-scroll-presentation
Jul 10, 2026
Merged

fix(ui): restore inset Help scrollbar and scroll-hint title#460
domcyrus merged 1 commit into
mainfrom
fix/help-scroll-presentation

Conversation

@domcyrus

@domcyrus domcyrus commented Jul 9, 2026

Copy link
Copy Markdown
Owner

The #459 rebase resolved the Help tab conflict by adopting the #452 pane-scrolling implementation wholesale, which changed the tab's presentation. This restores the original look and feel on top of the shared scrolling infrastructure:

  • Panel title reads Help · ↑/↓ scroll while the text overflows, plain Help when it fits.
  • The scrollbar sits one column inside the panel border so the border line stays intact, inset one row top and bottom to clear the title row and rounded corners (previously it was drawn over the border itself).
  • Right padding keeps the text clear of the blank gap + scrollbar columns, matching the Overview table arrangement.
  • The mouse wheel scrolls three lines per tick instead of one; the Help text is a long static page and the single-line step feels sluggish there.

Key handling is unchanged: it stays on the shared try_handle_pane_scroll helper and PaneScroll state from #452. The help_tab snapshot is regenerated accordingly.

The gradient-graphs rebase (#459) adopted the pane-scrolling Help tab
from #452 wholesale, losing the original presentation:

- panel title reads 'Help · ↑/↓ scroll' while the text overflows
- scrollbar sits one column inside the border (border stays intact),
  inset a row top and bottom to clear the title and rounded corners
- right padding keeps the text clear of the gap + scrollbar columns
- wheel scrolls three lines per tick instead of one

Key handling stays on the shared try_handle_pane_scroll helper and the
PaneScroll state introduced by #452.
@domcyrus domcyrus merged commit b7993f1 into main Jul 10, 2026
21 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.

1 participant