Thread Thumbnail by AddonsLab [Paid] 3.0.0 Beta 1

[Robert9]

1. Now all addons are deactivated.
2. Thread thumbs and lib are activated. Still same problem.

---
Next try: all addons deactived. Problem away.

 

[Robert9]

Conclusion: The problem is still there also, when i have only this addon (and the lib) active.
I will try to upload everything one more time. Maybe someone has an idea where the problem could be located.
I guess it is an old js somewhere. But which JS could be the right ones to look for?

 

[AddonsLab]

Conclusion: The problem is still there also, when i have only this addon (and the lib) active.
I will try to upload everything one more time. Maybe someone has an idea where the problem could be located.
I guess it is an old js somewhere. But which JS could be the right ones to look for?

Thank you for the report. We have already the issue and we are working on fixing it. It is caused by some conflict with XenForo JS itself, which, when deciing which attachments should be removed, simply queries all "checkbox" type inputs in the container. Our checkbox counts as "checked" as well and the attachment gets deleted. We are changing our checkbox to a radio that should resolve the problem.

The fix will be released here asap.

Thank you!

 

[Robert9]

Your addon runs in two client's forums without a problem (after we have changed something. Unfortunatley nobody remembers what it was)

 

[AddonsLab]

AddonsLab updated Thread Thumbnail by AddonsLab with a new update entry:

Bug-fix: All attachments are deleted when deleting a single attachment in a post

The release fixes a bug that would cause deleting all attachments in a post when one of them gets deleted. This was related to a conflict with XenForo core JavaScript, that would consider the checkbox meant for the thumbnail selection as a checkbox for deletion of attachments. We have updated the UI to use Radio buttons, and clicking a radio button one more time will unselect it, just in case you need to disable manual selection of any thread thumbnail.

The new version is available for all...

Read the rest of this update entry...

 

[beerForo]

What about the other 2 bugs and the custom thumbnail size?

 

[Robert9]

The size can be changed by some changes in .less, if i remember right.
It is a little bit tricky, because you have the search results, where you can change all sizes or only the threads or change just nothing.

Also there are list with a different html code for the icon-cell; so you need to change this cell also.

Then you need to decide where you change it and in which size. In some cases 72*96 are really nice, but then the avatar-text-icons look strange.
So maybe better to stay with 72*72 for example, depending on your content.

When i have the next client (and you really need it and if i remember it) i can write down the steps.

 

[beerForo]

The size can be changed by some changes in .less, if i remember right.

He said it was coming in the next version is why I asked.

 

[beerForo]

If this was just a quick fix for that one serious issue it should have been called a Patch.

 

[Robert9]

Oh, ok. It will be exciting to see where and which sizes are changed, because we have some more pages, where this can happen.
Also there are pages without a replacement.

 

[beerForo]

Here's the bugs:

Bug Report

• Enable only in forum view

If checked, thread thumbnail will be enabled only in the forum view. All other locations like New posts, Latest posts widgets, etc.will not show the thumbnail.

• The TT still shows in search results. To reproduce, go to a user profile and click to see their threads. You see a mix of avatars a TTs.
• Also "typo," no space after etc.

Bug Report

YouTube links with starting positions cause an image not found error for the TT. So when they choose to share and chose a "starting point," the link looks like this xxxxxxxxx:120 but it does not generate a TT and you get the red X.

I just had this happen again today.

Anyway, we will add a CSS class that can be used to target only the threads, that have thumbnails enabled in their forum.

This is the size option to be added. I stand corrected as in it did not say next version.

 

[AddonsLab]

Hello, @beerForo

Yes, this is a quick fix for a serious issue, that could cause content being deleted unexpectedly, that's why we have released it asap. We don't use Patch level releases, we just release minor upgrade in version "x.x.X" for bug-fixes and change version to x.X.x for newly implemented versions. Anyway, we will implement the bugs you reported and add the CSS class needed and release the next version within coming several hours.

Thank you!

 

[beerForo]

Sounds great! I also had this suggestion. In this scenario only an avatar shows when the TT could be from the unfurl. I have only had this happen once so far but perhaps it could be implemented in this version or later. Thanks!

Suggestion:
Since you are working on a new version would it be possible to grab the image from an unfurl if only an unfurl exists? Since my users post videos, they sometimes post unfurls since the website may not be supported for embedding on XF. But there is an image in the unfurl. Would you be able to set that as TT?

Example:

Shapes and Sizes

Unexpectedly unsettling images of Texas freeways, urban traffic patterns, and highway construction, rendered extraordinary by using two different cameras, on...

filmfreeway.com

 

[AddonsLab]

AddonsLab updated Thread Thumbnail by AddonsLab with a new update entry:

Bug-fixes and minor enhancements

• Bug-fix: YouTube thumbnail not detected if embedded URL has starting point
• Bug-fix: Thubmnails appear in search results even though they are activated for forum view only
• Added class "has-thumbnail" to the container row to apply specific CSS to rows with thumbnail

The new version is available for all licensed customers at

Login - AddonsLab

Thank you!

Read the rest of this update entry...

 

[AddonsLab]

Sounds great! I also had this suggestion. In this scenario only an avatar shows when the TT could be from the unfurl. I have only had this happen once so far but perhaps it could be implemented in this version or later. Thanks!

Sure, we will implement it soon.

 

[beerForo]

@beerForo we have just replied in your ticket. It seems re-importing the add-on fixes the issue. I can't be absolutely sure there is no coding issue/conflict there, but it is also possible to installation issue, if installation process somehow does not import class extensions fully.

We will confirm here once again if we manage to reproduce the issue.

Not fixed

Oops! We ran into some problems.
Please enter a value for the required field 'attachment_id'.

 

[AddonsLab]

Not fixed

Oops! We ran into some problems.
Please enter a value for the required field 'attachment_id'.

Can you please verify the add-on is active in the list of add-ons? Any errors from the validator? Last time the error seems to have fixed when we re-imported the add-on, yes? Unfortuantely, we don't reproduce the issue locally so it is hard to troubleshoot the problem further.

 

[AddonsLab]

Sounds great! I also had this suggestion. In this scenario only an avatar shows when the TT could be from the unfurl. I have only had this happen once so far but perhaps it could be implemented in this version or later. Thanks!

Just to clarify this feature. When a URL is shared, XenForo automatically embeds it into the page as a block, showing a thumbnail for it, its title and snippet from the page. Do you want thread thumbnail to use this automatically fetched thumbnail is available? Should this work with any website, or should be somehow configured which links will be considered as valid to detect the thumbnail?

 

[beerForo]

Can you please verify the add-on is active in the list of add-ons? Any errors from the validator? Last time the error seems to have fixed when we re-imported the add-on, yes? Unfortuantely, we don't reproduce the issue locally so it is hard to troubleshoot the problem further.

I replied to the ticket with info.

 

[beerForo]

Just to clarify this feature. When a URL is shared, XenForo automatically embeds it into the page as a block, showing a thumbnail for it, its title and snippet from the page. Do you want thread thumbnail to use this automatically fetched thumbnail is available? Should this work with any website, or should be somehow configured which links will be considered as valid to detect the thumbnail?

It would be great if it used this fetched image ONLY IF there was no other image. For example I use TT for a video forum. So people post YouTube embeds. However once in awhile the video they are linking to is on a website that XF does not embed. So instead of showing a video embed it shows an unfurl. And in the forum thread list it shows the avatar. In this case it should show the fetched image in the unfurl as the TT. If there was an embed, or attachment, then no.