@Madchatthew said in TNG + Nodebb:
you have to try and use duck tape and super glue to change something to make it do what you want it to do
I couldn’t have put that better myself.
Thats work. Many thanks for resolve
Thanks for the information, I understand better.
Would have a working solution other than mine for this :
@DownPW is that still an issue? I can’t replicate that on your dev site?
Besides, my solution above is only partial. It turns out that sometimes I can no longer click to reply to the post via quick Reply. Nothing happens when I click to enter text
cache maybe. I have delete the bugfix because not bug fix
@DownPW said in Threading support for NodeBB:
cache maybe. I have delete the bugfix because not bug fix
Yes, I can’t reproduce this.
I mean bug is present but my bug fix don’t work properly.
test to delete my cache and tell you
@phenomlab not cache
@DownPW PM me the link to the post.
actually, I noticed that it’s not on all the posts.
it seems that these are the posts where the last response is that of the user (selfpost)
DONE
@DownPW I’ll have a look at this later
no problem.
I have resolve this bug with change/replace this CSS CODE (I have just delete transform
directive) :
li.pt-4.self-post:not(.self-post .topic-owner-post).threaded {
/*transform: translateX(0px) !important;*/
transition: transform 0.3s ease !important;
background: var(--bs-body-bg);
background: var(--bs-body-navbar)!important;
padding-right: 30px;
margin-left: 0rem;
padding-bottom: 0rem !important;
}
I don’t know if is a goodfix or not.
@DownPW yes, was just typing that!
ha ha
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.
Mark hi,
I dont have this code in my js functions.
@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);
Updated git for above change