fix(soup): navigation scroll weird on small heights

[dev-rb]

Removes the offset scroll logic from soup navigation

[github-actions]

Preview: https://rahul-fix-soup-remove-context--lalh9a.preview.macro.com/app (0f7285d)

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2af55a19-4836-4e04-9b26-9ea13c0c06b1

📥 Commits

Reviewing files that changed from the base of the PR and between 5486f19 and ae108b4.

📒 Files selected for processing (1)

• js/app/packages/app/component/next-soup/soup-view/use-soup-navigation-hotkeys.ts

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Streamlined hotkey navigation scrolling in list views for more predictable and consistent behavior when navigating through items using keyboard shortcuts.

Walkthrough

Simplified the hotkey navigation's scroll behavior by removing context-offset computation, item-size-based spacing adjustments, and conditional index adjustments. Scrolling now targets the exact destination index with align: 'nearest'.

Changes

Cohort / File(s)	Summary
Navigation Hotkey Behavior js/app/packages/app/component/next-soup/soup-view/use-soup-navigation-hotkeys.ts	Removed context-offset computation, item-size-based scrolling spacing, and negative offset handling near the top boundary. Scrolling now uses exact destination index targeting with align: 'nearest', eliminating conditional index adjustments.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}