Yeah, as above, most thumbnailing logic is now in Discourse. I actually override to loosen the inclusion requirement slightly, but it still may be stricter than before.
I’m wary of overriding any more and really the ‘looseness’ of the criteria for considering something worthy of thumbnailing should now be a core issue. TLP should really just be a customer of that data.
I get it it now depends on Discourse behavior. Thanks for the help.
Just telling my status on this. Still struggling.
I’ve tried to normalize img html to include only src attribute since it seems to be ignored by discourse. For example usually a post from WP would start to contain this (here I’ve already cleaned the html, no attributes except src). The criteria you’re talking about does not seems to be related.
Then rebaking (so image are downloaded) from the console would work only for some topics.
Typically the topic where I’ve extracted the HTML above is ignored. Whatever rake posts:rebake or in rails post.save! ; post.rebake! ; the topic is ignored. I can’t see a reason why it fails, it has the same html as the others topics which worked.
However, if I edit manually from composer without changing the content and saving, then the image is extracted/downloaded. There are too much topics to edit manually.
Do you have an idea how you could achieve the same saving as composer does with rails console?
Note: the max old days option is set to the maximum value
That’s a very interesting, if frustrating, difference in behaviour. Can I suggest you post that as a Support Topic as that’s not really in scope of the plugin?
Got this bug this morning, could not find any solution and rolled back to last backup. Tried to change image and got again Errors 500/exception/ISE etc.
btw, no saying this plugin is faulty (error implied some s3 stuff). Just that using that particuliar image as thumbnail caused some weird behavior from discourse. Maybe it is, maybe not. This is first time it happened to me after many weeks using your plugin with full satisfaction, pretty weird
Thumbnail generation has moved to Discourse Core now. What TLP was doing to create thumbnails is now mostly done by Discourse. More and more TLP is just a presentation layer.
My guess is they decided to create a Theme Gallery and that required thumbnails (because presenting too many images at their original resolution would have made the page very slow). And it kind of went from there …
I think we should see this as a really positive move by Discourse btw. It’s great that the platform can now generate, store and deliver thumbnail data to the Topic List more or less out of the box without the need for a plugin.
In the long run that’s going to make the platform more attractive and help stability.
After upgrading to the last version of Discourse and TPL, we see glitches in the tiles. Some of them render properly as tiles, some don’t and then we get overlapping boxes, some content hidden. You can see for yourselves at https://eqlzr.org (you may need to scroll down to see it).
The browser console explains what is going on:
Layout was forced before the page was fully loaded. If stylesheets are not yet loaded this may cause a flash of unstyled content.
I don’t know whether this bug belongs to TLP or Discourse core.
Your site is rendering perfectly for me with both Chrome and Safari. But I can see some kind of issue in Firefox, interesting. I will take a look in more detail when I have time.
NB any issue with javascript can break the layout. This doesn’t have to be from TLP, it could be from any add-in or Discourse core as javascript ‘crashing’ will prevent TLP from finishing the layout.
Hey Robert, I’m testing out the TC on a different small site and having issues with it. I don’t have the TLP plugin installed, just the helper plugin. The Topic Thumbnails TC is working as expected which I have set for the default neutral theme on this site, I have TLPTC set for the dark theme so you can see what’s happening https://community.exploringexposure.com/ I’m on the latest of everything.