Adding a banner to chat messages

[DownPW]

Thank you very much @phenomlab , It’s very useful ^^

I just add a fa icon information to the code like this

$('[component="chat/message"]:last-of-type').append('<div id="chatbanner"><i class="fa fa-fw fa-circle-info link-primary" aria-hidden="true"></i> This session is for private discussion only between the chosen participants. Please do not place support requests here and use the forum categories instead.</div>');

On the other hand, we French use a lot of accent in our sentences like the word “j’aime” and if I put it here, the code does not work. So I found a sentence or I don’t put

I also wonder if this code will work on the new nodebb public chat room feature. I think yes but maybe we want to use a room to help live, so I think we will have to target this special room in the code to display another message (or none)

[phenomlab]

@DownPW said in Adding a banner to chat messages:

On the other hand, we French use a lot of accent in our sentences like the word “j’aime” and if I put it here, the code does not work. So I found a sentence or I don’t put

The reason for this is that the apostrophe is a system symbol, which means it needs to be escaped so that the text is read as was intended, and not as a system command. For example.

The below text

“This is an example paragraph that would not work if used in the traditional way. It’s going to be read incorrectly as code when it isn’t which means you’ll get an error instead of the text you actually wanted.”

Would need to be entered as

\"This is an example paragraph that would not work if used in the traditional way. It\'s going to be read incorrectly as code when it isn\'t which means you\'ll get an error instead of the text you actually wanted.\"

The usage pattern is easy to see, but if you want to play with potential combinations, use the site below

https://onlinestringtools.com/escape-string

@DownPW said in Adding a banner to chat messages:

I also wonder if this code will work on the new nodebb public chat room feature. I think yes but maybe we want to use a room to help live, so I think we will have to target this special room in the code to display another message (or none)

It should work in the same way. It will likely need to be extended to support new (multiple) targets, but will work nonetheless.

[DownPW]

Can you think you can update the code for v3.3 to target a specific room or several room?

[phenomlab]

@DownPW yes, that should be possible. I’ll have a look later on today.

[DownPW]

Ha yes very cool

[phenomlab]

@DownPW In my testing, this works already.

EDIT: As per PM discussion, I see what you’re looking for. This would effectively provide that result. It relies on the name of the room to display the relevant message - from the code example below, you can modify this to suit your needs

// Chat message banner
function chatBanner() {
    var roomName = $("h5[component='chat/header/title']").text().trim();
    if (roomName === "Test") {
        var banner = '<div id="chatbanner">This message will fire for chat rooms with the title of "Test"</div>';
    } else {
        var banner = '<div id="chatbanner">This session is for <strong>private discussion only</strong> between the chosen participants. Please do <strong>not</strong> place support requests here and create a <a href="#" onclick="app.newTopic();">new topic</a> instead.</div>';
    }
    return $('[component="chat/message"]:last-of-type').append(banner);
}

$(window).on('action:chat.loaded', function(data) {
    $(document).ready(function() {
        chatBanner()
    });
});

[phenomlab]

As requested by @DownPW (plus the samples above), below is some updated code that will also fire on empty rooms

// Chat message banner
function chatBanner() {
    var roomName = $("h5[component='chat/header/title']").text().trim();
    var bannerContent;
    if (roomName === "Testing 2") {
        bannerContent = '<div id="chatbanner">This message will fire for chat rooms with the title of "Testing 2"</div>';
    } else {
        bannerContent = '<div id="chatbanner">This session is for <strong>private discussion only</strong> between the chosen participants. Please do <strong>not</strong> place support requests here and create a <a href="#" onclick="app.newTopic();">new topic</a> instead.</div>';
    }
    var chatMessagesContainer = $('[component="chat/messages"]');
    var existingMessages = $('[component="chat/message"]');

    if (existingMessages.length === 0) {
        // If there are no messages, append the banner to the messages container
        chatMessagesContainer.append(bannerContent);
    } else {
        // If there are messages, add the banner after the last message
        existingMessages.last().after(bannerContent);
    }
}

$(window).on('action:chat.loaded', function(data) {
    $(document).ready(function() {
        chatBanner()
    });
});

You obviously need to specify the room names you are using (in the case of the sample above, it’s “Testing 2”). I’ve also restructured the code so that the following enhancements apply

Optimize Selector: Instead of using the [component="chat/message"] selector, which might be slower, I’ve decided to use classes (or where available, ID’s). This should improve performance, especially if the chat interface becomes more complex.

Appending the Banner: The previous code attempted to append the banner to the content of the last message - seeing as I’ve modified the code so that a primary condition exists first where if there are no messages in the room, the banner will still be appended (it wasn’t before). This means that the banner is no longer an independent element, but incorporated into the message stream which should make it faster.

[DownPW]

Hmmm tested your new code but I have the same banner message now

[phenomlab]

@DownPW Seems to work fine in my extensive testing

[DownPW]

Ok so there is a problem on my side. Sorry, test it again

[phenomlab]

@DownPW let me know how you get on.

[DownPW]

Work now with empty chat room, Thank you
But they appear at the very bottom of the chat :

Just see that the banner is displayed even after a system event :

component="chat/system-message"

System events should be considered as a separate message for thebanner message to be displayed below.

[phenomlab]

@DownPW did the chat banner work in the way you expected here?

[DownPW]

@phenomlab said in Adding a banner to chat messages:

@DownPW did the chat banner work in the way you expected here

A little bit. See hère:

@DownPW said in Adding a banner to chat messages:

Work now with empty chat room, Thank you
But they appear at the very bottom of the chat :

Just see that the banner is displayed even after a system event :

component="chat/system-message"

System events should be considered as a separate message for thebanner message to be displayed below.

[phenomlab]

@DownPW I think this is what you want - let me know

in this case, the code has to be modified so it looks like the below

// Chat message banner
function chatBanner() {
    var roomName = $("h5[component='chat/header/title']").text().trim();
    var bannerContent;
    if (roomName === "Testing 3") {
        bannerContent = '<div id="chatbanner">This message will fire for chat rooms with the title of "Testing 3"</div>';
    } else {
        bannerContent = '<div id="chatbanner">This session is for <strong>private discussion only</strong> between the chosen participants. Please do <strong>not</strong> place support requests here and create a <a href="#" onclick="app.newTopic();">new topic</a> instead.</div>';
    }
    var chatMessagesContainer = $('[component="chat/system-message"]:last-of-type');
    var existingMessages = $('[component="chat/message"]');

    if (existingMessages.length === 0) {
        // If there are no messages, append the banner to the messages container
        chatMessagesContainer.first().after(bannerContent);
    } else {
        // If there are messages, add the banner after the last message
        existingMessages.last().after(bannerContent);
    }
}

$(window).on('action:chat.loaded', function(data) {
    $(document).ready(function() {
        chatBanner()
    });
});

This will place the banner after the last system message and before any of the messages themselves. If there are existing messages in the room, the banner will be placed at the footer of the last message so it is seen each time.

[DownPW]

Thanks Mark, you are a champion

Test this code ASAP

[DownPW]

I have tested now this last code, It’s clearly better, Thnks you again.
Just one things mark @phenomlab, the banner don’t appear if there is empty (no message) :

[phenomlab]

@DownPW Does it appear if you reload the page?

[DownPW]

@phenomlab nope

[phenomlab]

@DownPW Very odd - I see this on your DEV server