Threaded post support for NodeBB

[phenomlab]

@DownPW yes, was just typing that!

[DownPW]

ha ha

[phenomlab]

Quick note here for anyone who is using the Threaded Posts code and has recently upgraded to 3.7.0 - only to find the threaded posts is now broken

This is being caused by the HTML structure changes that 3.7.0 makes in the DOM, meaning the element we used to target pre this release no longer exists. To resolve this, locate the line (within the threaded JS function)

$('.topic .sticky-tools ul.list-unstyled').append(threadView);

And replace it with

$('.topic .sticky-tools .thread-tools').append(threadView);

That should make it re-appear.

[cagatay]

Mark hi,
I dont have this code in my js functions.

[phenomlab]

@cagatay Yes, because you have legacy code. The line you have is this

$('.topic .sticky-tools ul [component="topic/browsing-users"]:last-of-type').append(threadView);

This will work fine because you are using the browsing users plugin, but should you decide to remove it, the threaded utility will cease to function. I’ve replaced that line with the below on your forum.

$('.topic .sticky-tools .thread-tools').append(threadView);

[phenomlab]

Updated git for above change

https://github.com/phenomlab/nodebb-harmony-threading/commit/14a4e277521d83d219065ffb14154fd5f5cfac69