Random News (D-NLS)

[Lynx]

Hi, Tiffy-Bean.

Please read the OP - you have to add a <div> in now where you want it to display. If you want it where it was before, simply go to Admin > Themes > Layout Templates > Forum Wrapper, scroll down until you see the $[content] variable, make a line above it and place this:


<div class="msg-rnf"></div>


I made that change so people can place it where they want it. There will be another update for it, so I'll see about adding in some options that you can just pick for auto-placement.

[Tiffy-Bean]

Oh perfect! Works. Sorry I missed that! Thank you love!

[Lynx]

Please let me know if you find any issues.

[Tiffy-Bean]

I know I've already said thanks Lynx, but I want to say thanks again.  This really is exactly what I needed and you were so prompt and nice to do it.  

I use it as a random "Did you know?" for my board where it explains aspects of the board and how to use it for newcomers, so I find it extremely useful.  I'll make sure to rate it five stars in the library.  Thank you for your time again.

[Lynx]

Glad you like it, Tiffy-Bean!

[Tiffy-Bean]

Attempting to hide the random news to guests. Tried this in my forum wrapper, but I'm not sure if that's how I should go about it. Any ideas?  Sorry about bugging you again. >.<

{if !$[current_user.is_member]}
<script>


$('msg-rnf').hide('msg-rnf');

</script>
{else}

{/if}

[Lynx]

Hi, Tiffy-Bean.

No need for the script tags. If it's in a template, just surround it with your {if}:


{if $[current_user.is_member]}
<div class="msg-rnf"></div>
{/if}


Only members will then be able to see the random news. If you need further help, just let me know.


Note: Remember that templates are theme-specific, so you'll need to make the edit above for each theme. I could probably do up a short script to use in your global header if you want it for all themes. I may need to see your forum so I can see exactly where you've got it placed - or you can post your Forum Wrapper template (feel free to PM me with it if you don't want to post it).

EDIT: Where do you have the actual <div> (<div class="msg-rnf"></div>) placed for the random news - in your Forum Wrapper template, another template or in a header / footer?

[Lynx]

Version 1.1.1 now in the library. OP updated with fix and new option.

[Tiffy-Bean]

I only use the one theme on my forum. I'm feeling a lil stupid that I didn't think of that! Thank you for being patient with me. Works perfectly!

[Boy_Wonder]

Heyas Lynx - apparently I've been let know by Peter that your code is making the proboards api complain. It was something about this so I'm quoting


Tell him... "event handlers require a function reference" so he can fix it

Does that help?

[Lynx]

Boy_Wonder,

Any idea, specifically, that is making the PB API complain? This is the first I've heard about this issue. Is it the proboards.on causing the issue?

[Boy_Wonder]

Nov 13, 2016 18:41:10 GMT -8 Lynx said:

Boy_Wonder ,

Any idea, specifically, that is making the PB API complain? This is the first I've heard about this issue. Is it the proboards.on causing the issue?

It's the afterSearch the () before the closing ) on the last line just needs removing

[Peter]

Nov 13, 2016 18:41:10 GMT -8 Lynx said:

Boy_Wonder ,

Any idea, specifically, that is making the PB API complain? This is the first I've heard about this issue. Is it the proboards.on causing the issue?

The reason it's the first is because most users don't use the console

But yeah, you need a reference passed, but you are "executing" your function call.

[Lynx]

Boy_Wonder,

Thanks for the info!

Nov 14, 2016 4:27:37 GMT -8 Peter said:

Nov 13, 2016 18:41:10 GMT -8 Lynx said:

Boy_Wonder ,

Any idea, specifically, that is making the PB API complain? This is the first I've heard about this issue. Is it the proboards.on causing the issue?

The reason it's the first is because most users don't use the console

But yeah, you need a reference passed, but you are "executing" your function call.

So, I just need to change this:

proboards.on('afterSearch',msgrnf());


to this:

proboards.on('afterSearch',msgrnf);


Is that correct?

[Boy_Wonder]

Yes that's correct XD Peter will correct me if I'm wrong