AJAX PM & Visitor Notifications - Bugs

[pi0tr3k]

Working, one problem tho.

When I had 2 notifications it was showing "Notifications: NaN"...

[Froomple]

Working, one problem tho.

When I had 2 notifications it was showing "Notifications: NaN"...

shows the same for 1 PM aswell

[Loget]

Ok, i enabled this thing now : http://i.imgur.com/vHyuO.png

And got this :
http://i.imgur.com/IBfnW.png
Now working..
You should work on the design probably, making it more noticeable

The design has been improved.

Thanks.

[Froomple]

The design has been improved.

Thanks.

its awesome now
Thanks !

[Hawk]

The issues have been resolved too (NaN and not able to close the notification).

I will leave this thread open so that if any other issues arise, you guys can notify us here.

Thanks.

[Loget]

We've added a sound alongside the notification so you won't miss it whilst in another browser tab.

[Long]

For me it's working, sound and all.

One small issue I noticed is that it updates the top notifications when a new pm is received but it doesn't remove it once it's been read, only on page refresh.

I'm talking about this,

[Mr Happy]

I enabled them both but I've visitor messages disabled. It should probably make the plugin check to see if the user has PM and Visitor Messages enabled before allowing ajax notification to be enabled and then present a warning message or offer the option to enable visitor messages.

[Hawk]

For me it's working, sound and all.

One small issue I noticed is that it updates the top notifications when a new pm is received but it doesn't remove it once it's been read, only on page refresh.

I'm talking about this,

I assume you open new PMs in new tabs? We made it so that PMs are read in the tab the notification is displayed, that way we wouldn't need to add an extra query. If you're on about the notifications while reading a PM - we'll have a look into it.

I enabled them both but I've visitor messages disabled. It should probably make the plugin check to see if the user has PM and Visitor Messages enabled before allowing ajax notification to be enabled and then present a warning message or offer the option to enable visitor messages.

We'll add a condition to it, thanks.

[ajinkya9]

You can also PM yourself if you want to test it.