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

Comment navigation duplicate requests. #2637

Closed
lukebp opened this issue Oct 14, 2021 · 0 comments · Fixed by #2638
Closed

Comment navigation duplicate requests. #2637

lukebp opened this issue Oct 14, 2021 · 0 comments · Fixed by #2638
Labels

Comments

@lukebp
Copy link
Member

lukebp commented Oct 14, 2021

The full comment section is being retrieved from the backend everytime you use the "see in context" feature and when you navigate back to the full thread using the "View All" button. There's no need for this. politeiagui already has all of the comments loaded.

This creates a bad UX since it adds in a delay when navigating between the full comment section and individual comment threads. The transition should be seamless.

The only time the full comment section should be fetched from the backend is if someone is navigating directly to the comment thread link and the comment section has not been previously loaded.

Example: https://proposals.decred.org/record/f7d9fc8/comments/4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant