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

Scroll #161

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

Scroll #161

wants to merge 15 commits into from

Conversation

jpan8866
Copy link

@jpan8866 jpan8866 commented Nov 14, 2024

Description ✏️

Features:

  • Auto scrolling
  • Ability for the user to stop auto scrolling
  • Ability to go back to auto scrolling by scrolling down, if output is still being rendered.
  • Scroll to bottom button (only shows when the content is not already scrolled to the bottom)
  • Fixed input box at the bottom after initial first prompt and enter.
  • Fixed indexing progress bar at the bottom.
  • Fixed New Session/Last Session buttons at the bottom
  • Added a fix to remove scroll bar buttons on StyledMarkDownPreview
  • Fixed model dropdown when the chat input is at the bottom
  • Fixed Add Context dropdown user experience (this makes it so it stays inside the container and doesn't create unnecessary scrollbars and overflow
  • Miscellaneous checks for whether the content in the sidebar has scrollable content, whether the user is scrolling, and other user interactions.

What changed? Feel free to be brief.

  • Bullet points are helpful.
  • Screenshots are helpful (if applicable).

Checklist ✅

  • I have added screenshots (if UI changes are present).
  • I have done a self-review of my code.
  • I have manually tested my code (if applicable).

cwilliams and others added 15 commits October 27, 2024 20:28
…yout

- Ensures output area stays within viewport bounds, preventing scrolling issues.
- Introduces AutoScrollContainer for smooth, controlled scrolling of the input/output history.
- Fixes an overflow issue in Markdown preview, improving readability.
- Refactors the layout for better visual consistency and user experience.
- Moves input area to a fixed position, allowing for easier access regardless of scroll position.
- Removes ChatScrollAnchor component, as its functionality is now handled by AutoScrollContainer.
index={index}
isLast={index === sessionState.history.length - 1}
isFirst={index === 0}
<AutoScrollContainer ref={containerRef} active={active}>

Choose a reason for hiding this comment

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

The only changes to this section are that I applied wrappers to the existing content and removed TopGuiDiv. This diff makes it look like there is a big change, but it is really just indentation.

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