Fix: Duplicates when click spam loading elements in FT (3/3)#9412
Fix: Duplicates when click spam loading elements in FT (3/3)#9412Shadorc wants to merge 4 commits into
Conversation
|
Can we also make "View N replies" shows some loading state specific text (better then hidden coz user can request replies for multiple comments at the same time (and maybe remove the toast for Getting comments replies if that's done |
|
"View N replies" is the only text in my three MRs that I don't hide, the behaviour is the same as before, the text is here while the replies have not loaded. |
|
Ya I don't need the text to be hidden unlike those "next page/load more" buttons |
Head branch was pushed to by a user without write access
|
I have added a loading message for "View n replies" and "Show more replies" |
|
I didn't include it in the tests because there are already so many, but it behaves similarly in the Posts tab |
Pull Request Type
Related issue
partially addresses #3541 (comments)
closes #3541 if all three MRs are merged
Description
Same remarks from #9383 apply here + this comment #9383 (comment).
Those MRs are independent and can be merged in any order.
I've slightly refactored
getPostCommentsInvidiousto better match similar methods.Screenshots
2026-07-12.14-34-25.mp4
New changes related to loading state
https://github.com/user-attachments/assets/68865337-3d93-4e32-b2a2-618089a798e1
Testing
More comments - Manual:
More comments - Automatic:
View replies - Single:
getCommentRepliesto make it easier)View replies - Multiple:
getCommentRepliesto make it easier)More replies: