nodebb-plugin-blog-comments icon indicating copy to clipboard operation
nodebb-plugin-blog-comments copied to clipboard

Reply/Quote comments

Open frandiox opened this issue 9 years ago • 3 comments

@psychobunny This implements reply/quote to blog comments. It also adds a label that scrolls to the replied comment (similar to NodeBB's one). This is meant to be a basic implementation. It would be better to use a scrolling animation rather than scrollIntoView, for example.

capture

(The font-size is different in the image due to some local CSS)

I included also a link to the user profile from username because on small resolutions the profile pictures are hidden (those are the links right now).

And by the way, I couldn't figure out how to access to <!-- IF posts.toPid --> (it can be a string or undefined) and {posts.parent.username}, so I had to create posts.isReply and posts.parentUsername. If you know a better way I can change it.

frandiox avatar Jan 12 '16 12:01 frandiox

It seems that a reply to a purged post contains parent object without parent.username inside, so it's possible to remove the "reply-label" from those posts. However, in the case of replies to deleted posts (not purged), I think there is no way to differentiate from normal replies. /api/topic/id/... returns deleted posts as well, but the blog comments uses a different request which does not proportionate them. I notice that in NodeBB itself a reply to a purged post will show the label without any content, what I guess it's a bug.

frandiox avatar Jan 13 '16 05:01 frandiox

I used this patch in https://github.com/revir/nodebb-plugin-blog-comments2 It works as I desired, thank you! But you seems to forget handling unlogined reply, but everything others is good :)

revir avatar Jan 15 '17 08:01 revir

@revir Thanks! That's true, I forgot to hide the buttons when it's not logged in, my bad. I will try to update the PR when I have some time. I think this is still useful for some users even though I doubt it will be merged to the main repo at this point :sweat_smile: For other users, live demo here.

frandiox avatar Jan 15 '17 09:01 frandiox