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

Combine ChitChat posts #105

Open
edwh opened this issue Nov 25, 2024 · 0 comments
Open

Combine ChitChat posts #105

edwh opened this issue Nov 25, 2024 · 0 comments

Comments

@edwh
Copy link
Member

edwh commented Nov 25, 2024

Take this example, caused by someone hitting enter before they've finished their whole post.

image

Ignore for the moment the rathole of enter-as-newline vs enter-as-send.

If we have adjacent chitchat posts from the same user within a short period of time then it would make sense to display them as a single post which combines them, with a line break between.

The relevant code is in pages/chitchat/[[id].vue around line 335. Need to make sure that the use of scroll-to in that page works, though.

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

No branches or pull requests

1 participant