[nodebb-plugin-shoutbox] Shoutbox plugin

xCausxn

Not working with lavender theme just letting people know

psychobunny

let me know what I got to do to fix that @Mr_Waffle

Schamper

works fine
Working fine for me.

bentael

we need to ditch the
$(document).bind('DOMNodeInserted', function(event) {
p.s. I have no idea how badly that performs but I would imagine it sucks.

it does suck - but you should see this, i've used it before, works like a charm (minus IE8)
http://jsfiddle.net/PkYb9/2/

EDIT: time for a jsfiddle/jsbin embed plugin

psychobunny

haha well since you just said my codez suck your opinion is irrelevant

kidding

anyways ajaxify.onchange is the best solution to detect what page we're on, needs to be implemented, I'll make an issue

Schamper

0.0.4 has been published.
Global changelog:

Fixed user box, now actually shows the online users
Make usernames bold
Added setting in AdminCP for the placement of the shoutbox
Added action in AdminCP to prune shouts marked as deleted
Added action in AdminCP to delete all the shouts from the database
Archive button is now implemented (So you can look back at your highly intellectual shouts, like mine!)
Overall refactoring to make everything better
Admins can now delete everyone's shouts
Probably some other stuff I forgot

bentael

@Mr_Waffle +1

xCausxn

+1 good work mate

xCausxn

@Mr_Waffle Any way to put the shoutbox in the sidebar?

Schamper

@xCausxn Not yet. @psychobunny was talking about a 'widget' feature he's planning where plugins can be designated to from the AdminCP, so you'll have to wait for that. To keep max compatibility I'm only supporting top and bottom of the page now.

Schamper

@StuartH said:

@mr_waffle I have the shoutbox installed but it says I must be logged in to post but I am logged in!! I logged out then in, cleared cache, on nodebb 2.1, I can break anything

I think I know what the issue is with this. When it's taking too long to execute some piece of code (don't know which) that's responsible for logging you in, you'll appear as not being logged in (also in the header). The shoutbox then checks if you're logged in, and hoozah, you're not -> message is displayed. A page refresh often fixes this.

Schamper

0.0.5 published
Changelog:

Deleting of shouts is fixed, other users can now see the real-time result of shout deletion (required refresh prior to this fix)
You can now edit your shouts (woo!), proper real-time updating this time!
A message is now shown if the shoutbox is empty to prevent confusion between an empty shoutbox and a non-loading one.

psychobunny

Hey @Mr_Waffle I feel like your shoutbox should just apply a hide class on the whole thing outright when logged out, since you're not allowing users to view the actual content anyways (just takes up space on your home page)

Schamper

@psychobunny said:

Hey @Mr_Waffle I feel like your shoutbox should just apply a hide class on the whole thing outright when logged out, since you're not allowing users to view the actual content anyways (just takes up space on your home page)

Good point.

Scuzz

shoutbox

Having this issue @Mr_Waffle
Posted over at BitBangers but i thought i'd post it here too as this is the more official thread.

Schamper

Saw it as well. You'll have to clear your cache since I updated the CSS + client side JS.

Scuzz

NodeBB and it's damn cache!

julian

@Scuzz not anymore! It's just the plugin's css that not using the cache buster, which I don't take responsibility for

psychobunny

Actually I think we should theme css can take advantage of cache buster but not plugins!

psychobunny

Plugin CSS should take advantage of Cache Buster · Issue #895 · NodeBB/NodeBB

This Should be minified along with theme.css imo, or at the very least have ?{cache-buster} appended to it

GitHub (github.com)