Skip to content

vBot item list scroll to bottom on new entry#81

Merged
divinity76 merged 1 commit intoOTAcademy:masterfrom
divinity76:vbot-item-scroll-to-bottom
Jan 20, 2026
Merged

vBot item list scroll to bottom on new entry#81
divinity76 merged 1 commit intoOTAcademy:masterfrom
divinity76:vbot-item-scroll-to-bottom

Conversation

@divinity76
Copy link
Copy Markdown
Collaborator

@divinity76 divinity76 commented Jan 10, 2026

previously when adding a new item to a large list, the list would scroll to the top each time.
For very long lists with a lot of stuff to add,
this becomes increasingly annoying and time-consuming for each new entry.
If PR #78 is accepted, large item lists are likely to become a lot more common (one entry for every garbage thing you do not want, allowing you to loot all the super-rare items you do not have the id for, or don't even know exist)
image

previously when adding a new item to a large list,
the list would scroll to the top each time.
For very long lists with a lot of stuff to add,
 this becomes increasingly annoying and time-consuming for each new entry,
 to scroll back to the bottom to add a new entry.
The amount of scrolling requires increase for every 4 new entries.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the user experience when managing vBot item lists by automatically scrolling to the bottom when items are added or updated. Previously, the list would scroll to the top on each change, which was frustrating for users managing large lists with many entries.

Changes:

  • Added scrollToBottom helper function that safely scrolls the container's scrollbar to the bottom
  • Integrated automatic scroll-to-bottom behavior in the setItems function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@divinity76 divinity76 merged commit 9775fc4 into OTAcademy:master Jan 20, 2026
6 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.

2 participants