#2340 closed enhancement (fixed)
Hierarchical replies enahancments
Reported by: | alex-ye | Owned by: | |
---|---|---|---|
Milestone: | 2.4 | Priority: | normal |
Severity: | normal | Version: | trunk |
Component: | Component - Replies | Keywords: | needs-patch |
Cc: | jmdodd@… |
Description
Hi, First day of testing the new bbPress feature "Hierarchical replies" for now I have some notes:
1- If I am not miss something, I can't see any sufferance between the 0 and 1 in the deep level setting.
2- I can't set the reply-to input to 0, seems something wring with bbp_update_reply_to() function.
3- I can't see any reason to show the reply-to link in the deepest replies ( I can't get the correct meaning in English :D )
4- The reply-to from title should be changed depending on the reply ID ( Reply To: %topic-title% - Reply %reply-id% )
that's all for now...
Change History (17)
#3
@
11 years ago
bug: the reply-to link shouldn't be shown in the archive pages like the user-replies, right?
#4
@
11 years ago
Please can you check the child replies CSS classes, The Odd/Even classes and could you mind adding 'bbp-parent-reply-%" class?
#9
in reply to:
↑ 2
;
follow-up:
↓ 10
@
11 years ago
Replying to alex-ye:
another bug need someone to confirm: the _bbp_replies_per_page option doesn't work in the topics anymore.
We've turned off topic pagination for threaded replies for now. We'll likely iterate on this more in future versions.
#10
in reply to:
↑ 9
;
follow-up:
↓ 14
@
11 years ago
Replying to johnjamesjacoby:
Replying to alex-ye:
another bug need someone to confirm: the _bbp_replies_per_page option doesn't work in the topics anymore.
We've turned off topic pagination for threaded replies for now. We'll likely iterate on this more in future versions.
Hmmm, sad news :(
The topic pagination is the most thing I use in my forums, How you sugget me to deal with a topic has more than 100 reply?
Topic replies are diffrent from post comments...
At least can you explain why the pagination will doesn't work?
#14
in reply to:
↑ 10
;
follow-up:
↓ 16
@
11 years ago
- Resolution set to fixed
- Status changed from new to closed
Replying to alex-ye:
At least can you explain why the pagination will doesn't work?
We haven't decided on which approach is best to take:
- Paginate based on root thread count
- Paginate based on total visible replies
- Some new hybrid approach
We're basically mirroring the P2 theme, in the sense that if you're asking for a visible hierarchy, you probably want as much of the conversation available as possible, without the concerns of splicing hierarchical replies mid-conversation.
I'm of the opinion that we either need to count the root-replies, and paginate by those, or dream up some new approach. 20 visible replies in a hierarchy isn't enough, and it poses the problem of what to do if the number of visible replies cuts off conversations.
Since this is a new feature, I want to see how our users react before putting false limitations on it.
Marking this ticket as resolved, since the issues raised in it are fixed. For more bugs or issues pertaining to threaded replies, please open single tickets for each issue.
Thanks for keeping an eye on this!
#16
in reply to:
↑ 14
@
11 years ago
Replying to johnjamesjacoby:
We haven't decided on which approach is best to take
I see now :), It is really confuse point to discuss!
Can you open ticket for this, so the users know it's a known bug?
Since this is a new feature, I want to see how our users react before putting false limitations on it.
I agree with you, but I would love to see this bug resolved in the final version since the both are important features.
Thanks you for everything :)
#17
@
9 years ago
Sanal Tur:http://sanaltur.com
another bug need someone to confirm: the _bbp_replies_per_page option doesn't work in the topics anymore.