Threaded comments can be useful for organizing sub-conversations within comments, but it can get out of hand quickly. A useful feature for flatcomments is to be able to specify a maximum depth of comments, per content type. So if it's the first thread, all replies to comments within that thread will be flattened. Here's an example:

Change this:

  • comment 1
    • comment 3
      • comment 7
    • comment 4
      • Comment 5
  • comment 2
  • comment 6

To this:

  • comment 1
    • comment 3
    • comment 4
    • comment 5
    • comment 7
  • comment 2
  • comment 6

Comments

dragonwize’s picture

dragonwize CreditAttribution: dragonwize commented
Status: Active » Closed (won't fix)

This can already by done through the theme layer. The indention of comments is actually created by design, usually CSS. All you have to do is theme all comments under the first layer the same way.

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Closed (won't fix) » Active

I think the OP meant differently. CSS changes the indentation, but can't change comment order.

dragonwize’s picture

dragonwize CreditAttribution: dragonwize commented
Title: Threaded flat comments » Limit threaded depth
Status: Active » Postponed

That would be a nice option. Though I am not entirely sure it belongs in this module. I will have to think about that. Feel free to express why you think it should or should not be in the flatcomments module.

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Postponed » Active

The purpose is to allow maximum N+1 levels of comments. With this module, N=0. However I don't see other module does that, and there is a need for N > 0.

The principe, when you reply to a comment, it check if the parent comment has a level > N, it will descend to level N and change the parent of the new-submitted comment to that comment. Let's take the example given by the OP, when we add cid7, if N=0 (current), its parent will be set to NULL, if N=1, its parent will be set to cid1, if N>=2, its parent will be kept unchanged (= cid3).

Flatcomments actually set pid=NULL to limit max level = 1, but it can do that in a more general way. Moreover, it requires just a small modification.

dragonwize’s picture

dragonwize CreditAttribution: dragonwize commented
Status: Active » Postponed

I understand what you are asking for and agree the feature would be great.

My disagreement is that extra options to for "threaded" comment configuration is not really in the realm of a module that is named "flatcomments" and gives extra options for "flat" comment configuration.

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Postponed » Active

Why we should implement this feature in the module?
- There is a real need: for forum-like discussion we need flat comment, ok. But for blog, we might need N-level threaded comment.
- It's easy and just a small add-on to this module
- If flatcomments doesn't support it, there will be another module does it (or clone, frankly). That module does of course all flatcomments does. => duplicate effort.

dragonwize’s picture

dragonwize CreditAttribution: dragonwize commented
Status: Active » Postponed (maintainer needs more info)

Please stop changing the status.

jcisio’s picture

jcisio CreditAttribution: jcisio commented

You need info. I give you you info, thus I'm expected to change the status (because it has changed, indeed). But I'm sorry if it causes a problem for you.

jcisio’s picture

jcisio CreditAttribution: jcisio commented

I need this feature again in another project and release a new module http://drupal.org/project/tlcomment.