Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/initial scroll button visibility #1978

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hexart
Copy link
Contributor

@hexart hexart commented Mar 10, 2025

  • Add one-time useEffect to detect if page loads with content not scrolled to bottom
  • Use setTimeout to ensure content is fully loaded before checking scroll position
  • Set showScrollButton visibility correctly for history sessions
  • Fix edge case where scroll button wouldn't appear when loading non-bottom-scrolled content

This improves user experience when viewing historical conversations by immediately showing the scroll-to-bottom button when needed.

hexart added 2 commits March 10, 2025 11:05
- Add one-time useEffect to detect if page loads with content not scrolled to bottom
- Use setTimeout to ensure content is fully loaded before checking scroll position
- Set showScrollButton visibility correctly for history sessions
- Fix edge case where scroll button wouldn't appear when loading non-bottom-scrolled content

This improves user experience when viewing historical conversations by
immediately showing the scroll-to-bottom button when needed.
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. frontend Pertains to the frontend. labels Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Pertains to the frontend. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant