Skip to content

fix: restore backup list scrolling#3

Merged
OneNoted merged 2 commits into
mainfrom
fix-backup-list-scrolling
Mar 19, 2026
Merged

fix: restore backup list scrolling#3
OneNoted merged 2 commits into
mainfrom
fix-backup-list-scrolling

Conversation

@OneNoted
Copy link
Copy Markdown
Owner

Summary

  • apply backup list scroll-window logic so the selected row stays visible
  • reset backup list scroll state when filters empty the view or the viewport has no visible rows
  • keep delete actions aligned with the visible selection in long filtered lists

Verification

  • cd tui && zig build test -Doptimize=ReleaseSafe
  • cd tui && zig build -Doptimize=ReleaseSafe
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7966f21292

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tui/src/views/backups.zig Outdated
@OneNoted OneNoted merged commit c5c5866 into main Mar 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant