Thread Watchers 1.2

[vVv]

Hmm...

Yes!! Love it!

 

[MattW]

Error

Please enter a valid callback method.

When attempting to upgrade to 1.1

I had this but was due to APC caching the old files, and not the new ones. Just cleared the cache and updated perfectly.

 

[Chris D]

Chris Deeming updated Thread Watchers with a new update entry:

Tweaks and fixes

• Watchers icon now floated to the right (consistent with other thread alerts).
• Watchers list now has "noindex" so will not be indexed by search bots.
• Text based link no longer placed into position using javascript
• Watchers alert now above the Thread Tools.

Read the rest of this update entry...

 

[vVv]

Chris Deeming updated Thread Watchers with a new update entry:

Tweaks and fixes



Read the rest of this update entry...

awesome! thanks man, installing in few minutes!

 

[DRE]

You added an alert? Like it tells you who watches your thread?

 

[Jeff Fuqua]

Wonderful idea. I just added this. However, it's not showing up as either a bar or text. I thought it might need to be a new thread but no luck there either.

The "add a poll" is there so wonder if there might be some kind of conflict.

EDIT: Looks like an update was added so re-installed and now all is good.

Thanks!

 

[Chris D]

You added an alert? Like it tells you who watches your thread?

No.

The blue horizontal bar with the glasses icon is an alert. At least it is described as such in CSS, just like the closed bar. It's css is ".threadAlert".

Wonderful idea. I just added this. However, it's not showing up as either a bar or text. I thought it might need to be a new thread but no luck there either.

The "add a poll" is there so wonder if there might be some kind of conflict.

Maybe there's no one watching the thread? I doubt it's a conflict as I have both those add-ons installed (you can probably see that in some of the screenshots) and it works fine.

 

[Jeff Fuqua]

It may have been no one was watching the one I viewed. Regardless, I uploaded the latest update and all is good.

 

[Shelley]

Chris Deeming updated Thread Watchers with a new update entry:

Tweaks and fixes



Read the rest of this update entry...

Excellent update Chris. Installed without any issues and displays as intended. Excellent work yet again.

 

[vVv]

Installed it, works perfectly... Nice, and thank you Chris!

 

[Veer]

Chris Deeming updated Thread Watchers with a new update entry:

Tweaks and fixes



Read the rest of this update entry...

Very nice update Chris, upgraded and it worked nicely. Thank you very much

 

[Luxus]

You added an alert? Like it tells you who watches your thread?

Considering that by default people automatically watch threads when replying I'm not sure it's a good idea to annoy people with a buck load of pointless alerts.

 

[Cory Booth]

I had this but was due to APC caching the old files, and not the new ones. Just cleared the cache and updated perfectly.

Ah! Thank you... New to APC and Memcache...

 

[Sheldon]

Finally got around to adding this.

Awesome work Chris, thanks.

Also, thanks for the image addition Shelley.

 

[fredrikse]

What a great add-on, Chris! Is is possible to implement some sort of permission control for this feature? I rather not show this information to non-members.

 

[Sheldon]

What a great add-on, Chris! Is is possible to implement some sort of permission control for this feature? I rather not show this information to non-members.

You could likely wrap it in some conditionals. I wouldn't know for certain what to use though, I have seen the code posted around the forums tons of times though.

 

[fredrikse]

I just thought of an idea. Wouldn't it be cool to have a feature like...

"Members who is watching this thread is also watching..."

This feature is common on for instance Amazon.

 

[fredrikse]

You could likely wrap it in some conditionals. I wouldn't know for certain what to use though, I have seen the code posted around the forums tons of times though.

I often use TMS to create some sort of permission control. That wouldn't really be a problem for me. But it always great to get features into the core code.

 

[Chris D]

I often use TMS to create some sort of permission control. That wouldn't really be a problem for me. But it always great to get features into the core code.

Because it is a basic conditional, I'd be inclined not to introduce any sort of permission control for it.

Quite simply, in thread_view_watchers template, add the code in red.

<xen:require css="thread_view_watchers.css" />

<xen:if is="{$watchers} AND {$visitor.user_id}">
	<xen:if is="!{$xenOptions.threadWatchersTextOnly}">
		<dl class="threadAlerts secondaryContent">
			<span class="alertIcon"></span>
			<dt>{xen:phrase threadwatchers_watchers}:</dt>
			<xen:if is="{xen:number $watchers} > 1">			
				<dd>{xen:phrase threadwatchers_this_thread_has_x_watchers, 'count={xen:number $watchers}', 'link={xen:link threads/watchers, $thread}'}.</dd>
			<xen:else />
				<dd>{xen:phrase threadwatchers_this_thread_has_x_watcher, 'count={xen:number $watchers}', 'link={xen:link threads/watchers, $thread}'}.</dd>
			</xen:if>
		</dl>	
	</xen:if>
</xen:if>

And in thread_view_watchers_text template:

<xen:if is="{$watchers} AND {$visitor.user_id}">
	<xen:if is="{$xenOptions.threadWatchersTextOnly}">
		<a href="{xen:link threads/watchers, $thread}" class="OverlayTrigger">{xen:phrase threadwatchers_watchers}: {xen:number $watchers}</a>
	</xen:if>
</xen:if>

 

[Allan]

French language