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

Automatic free scroll toggling and scroll to bottom floating button. #60

Closed
wants to merge 1 commit into from
Closed

Automatic free scroll toggling and scroll to bottom floating button. #60

wants to merge 1 commit into from

Conversation

jondwillis
Copy link
Contributor

#59

@vercel
Copy link

vercel bot commented Apr 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nextjs-chatgpt-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 6, 2023 10:49pm

@enricoros
Copy link
Owner

I love this! Need to do some cleanup for mergind this (somehow the PurposeSelector is gone, and the buttons appear to the far right of the screen (very right, see below).

But this is brilliant!
image

@enricoros
Copy link
Owner

I've been at this twice now, and every time I get stuck for some reason or another. The looks are a bit off (item is on the side, it's a toggle, etc). I love the experience of this feature, but we're not there yet.

enricoros added a commit that referenced this pull request Apr 19, 2023
The drawback is that it's not smooth scrolling when going down.
The positive is that this is a fully CSS implementation.
@enricoros
Copy link
Owner

I've implemented a pure CSS solution to this (turns out that with flex-direction: 'column-reverse', it defaults to sticking to the bottom, unless the user has moved the scrollbar. The browser does the implementation basically.

However lost the smooth scrolling that was there before (each new line is immediately displayed). Removed the "Free Scroll" option too. Big win.

@enricoros enricoros closed this Apr 19, 2023
@PaRaD1SE98
Copy link

In mobile Safari, it is not going down properly for some reason.

RPReplay_Final1683067370.mp4

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.

3 participants