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

Live updates to comment threads #791

Open
huumn opened this issue Feb 7, 2024 · 0 comments · May be fixed by #889
Open

Live updates to comment threads #791

huumn opened this issue Feb 7, 2024 · 0 comments · May be fixed by #889
Assignees
Labels
difficulty:hard feature new product features that weren't there before

Comments

@huumn
Copy link
Member

huumn commented Feb 7, 2024

This is something we've been wanting to do for awhile.

Goal: When viewing a post, display new comments as they come in on a post. The result would make comment threads slightly more lively destinations and chat-like.

Things to consider:

  • we don't want the layout to shift automatically when this happens, so new comments should probably just result in an absolutely positioned button offering to display new comments in the branches where they'd appear
  • perhaps not an MVP feature, but it'd be nice when a tab is left open to a post, its appearance changes when there are new comments (like how notifications add a red dot to the left corner, perhaps new comments can make the right corner blue)
  • we'd prefer to attempt this by polling rather than using web sockets
    • at some regular interval, you'd just query the backend for comments with a creation or update time > last check
    • it'll be paramount that this check is done efficiently
@huumn huumn added feature new product features that weren't there before difficulty:hard labels Feb 7, 2024
@mzivil mzivil self-assigned this Feb 22, 2024
@mzivil mzivil linked a pull request Feb 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty:hard feature new product features that weren't there before
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants