Skip to content

Threaded chat support for NodeBB

Let's Build It
35 3 16.1k 1
  • wrote on last edited by DownPW
    #26

    Ok chief 🙂

    – Result :

    image.png

  • wrote on last edited by DownPW
    #27

    hello my friend @phenomlab continue t otest and debug, I see this bug.

    When you edit your post and save it, you lose the display (or the post simply disappears) and you have to refresh the page to see it again and display it correctly

    EDIT:

    • in public can :edit post lose display/layout
    • In private chat : edit post lose the post

    public :
    blink2.gif

    private:
    blink3.gif

  • hello my friend @phenomlab continue t otest and debug, I see this bug.

    When you edit your post and save it, you lose the display (or the post simply disappears) and you have to refresh the page to see it again and display it correctly

    EDIT:

    • in public can :edit post lose display/layout
    • In private chat : edit post lose the post

    public :
    blink2.gif

    private:
    blink3.gif

    wrote on last edited by
    #28
  • wrote on last edited by DownPW
    #29

    this :

    $(window).on('action:chat.onMessagesAddedToDom', function(data) {
            	$(document).ready(function() { 
        	    console.log('hook triggered'); 
            threadedChat();
    });
    });
    

    Seems you have add it on the function.js on github no ?

  • this :

    $(window).on('action:chat.onMessagesAddedToDom', function(data) {
            	$(document).ready(function() { 
        	    console.log('hook triggered'); 
            threadedChat();
    });
    });
    

    Seems you have add it on the function.js on github no ?

    wrote on last edited by phenomlab
    #30

    @DownPW Yes, but check the git link - you need to modify public/src/client/chats/messages.js as below

    https://github.com/NodeBB/NodeBB/commit/a7517d886f83703dc158c0e2327542156b539eb6

    101be178-1a40-4ff6-bb8b-aa5dde5aec0b-image.png

  • wrote on last edited by DownPW
    #31

    ha ok I misunderstood that. I thought it was just to add in the Javascript code.

    So we have to create the hook manually until 3.5.0

    Do you think this will solve the problem?

    Test ASAP

  • ha ok I misunderstood that. I thought it was just to add in the Javascript code.

    So we have to create the hook manually until 3.5.0

    Do you think this will solve the problem?

    Test ASAP

    wrote on last edited by
    #32

    @DownPW said in Threaded chat support for NodeBB:

    Do you think this will solve the problem?

    Yep - I have the patch and cannot replicate the issue on here 🙂

  • wrote on last edited by
    #33

    yep confirmed. All ok with the hook

  • wrote on last edited by
    #34

    Better like this : add shadow and border-left on self answer 🙂

    image.png

  • Better like this : add shadow and border-left on self answer 🙂

    image.png

    wrote on last edited by
    #35

    @DownPW said in Threaded chat support for NodeBB:

    Better like this : add shadow and border-left on self answer

    Of course - you style to your own requirements and taste 🙂 I’ll commit that CSS we discussed yesterday also

  • phenomlab phenomlab forked this topic on

Related Topics