#43429 closed enhancement (fixed)
Add "In response to ..." before threaded comments in comment feed
Reported by: | zodiac1978 | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Comments | Keywords: | ux-feedback has-patch commit |
Focuses: | Cc: |
Description
Imagine this in your feed reader:
Comment 1: WordCamp Miami was awesome!
Comment 2: I do not learn anything. It was boring ... :(
Comment 3: YES, it was!
But Comment 3 is a reply to Comment 1. But we don't see it.
I am thinking about making it to something like that:
Comment 1: WordCamp Miami was awesome!
Comment 2: I do not learn anything. It was boring ... :(
Comment 3: (In response to *Comment 1*.) YES, it was!
There already some places in WordPress which are doing that:
Attachments (2)
Change History (16)
This ticket was mentioned in Slack in #design by karmatosed. View the logs.
6 years ago
#6
@
5 years ago
- Severity changed from minor to normal
Read the Slack "discussion" and wanted to give more feedback, but I don't know what is meant by this feedback from @karmatosed :
What do people think?
Is it a question about the user and therefore a rhetorical UX question? Or is it something directed to me? Do you think this is an absurd edge case which should be ignored?
mukesh27 wrote:
It can be achieved using plugin as per user said
Yes, this plugin is from me, but if WP is interested this could be merged to core. If not, please close as Wontfix. If yes, I can provide a patch for it based on the plugin.
joyously asked:
It's a feed, isn't there a format already for it? Does it depend on what you use to view the feed? Would it change what the feed reader software is expecting?
I don't know any feedreader which can show threaded comments. All feedreader show the feed in plain chronological (or reverses chronological) order. No threading. And therefore this addition would make sense IMHO. The RSS spec has no threading at all if I am right.
#7
@
5 years ago
@zodiac1978 sorry that was confusing for you to read out of context, let me clarify. Every week the design team runs a triage session. This came up during it. I was offering up the issue for discussion to the group attending. I wasn't bringing any judgement as it was something everyone was able to give insight on.
#8
@
5 years ago
- Keywords dev-feedback added
Thanks for the clarification @karmatosed - I thought it was rude, because for me it looks like you were questioning the significance of the ticket. Fortunately this wasn't the case.
I am still interested if this could be something for core. Does anyone have feedback for me about the mentioned plugin solution?
#9
@
5 years ago
- Keywords has-patch added; needs-patch dev-feedback removed
- Milestone changed from Awaiting Review to 5.3
- Owner set to SergeyBiryukov
- Status changed from new to accepted
Adding In reply to %s
to child comments in comments feed seems like a quick win.
Let's do this, along with #43805.
#10
@
5 years ago
- Milestone changed from 5.3 to Future Release
With version 5.3 Beta 1 releasing shortly, the deadline for enhancements is now passed. This is being moved to Future Release
.
#12
@
5 years ago
- Keywords commit added
Hi @SergeyBiryukov
Thanks a lot for your work on the patch. I've just added a .2.diff one to edit the docblock part changing 5.3.0 for 5.4.0. I've also tested the patch and I confirm it behaves as expected. I believe, just like #43805 it's ready to be committed.
Working plugin solution:
https://github.com/Zodiac1978/threaded-comment-feed-enhancer/