Topic List Previews (legacy)

Thanks for the fix! I just set a resolution of 4, and almost every tile is now loading 400x400 size thumbnails. So that’s working fine. :clap:

The only exceptions are a few posts with an original image upload of less than 800x800 but more than 400x400. These posts are loading the 200x200 size thumbnail instead of the available 400x400

1 Like

The new algo uses rounding. You probably have to accept some will be switched up or down. This is so I can generalise ‘resolution level’ across sets of different length. The issue might be I’m treating each division as equal but in reality it should be non linear.

Sorry can’t spend more time on this at mo. I may improve it in the future.

2 Likes

Sounds good, thanks for the fixes. I’m happy that we’re loading 400x400 sizes now to reduce our page loads. :+1:

2 Likes

@merefield, first off… thanks for the amazing plugin :pray:

I just wanted to also report that the “topic list default thumbnail fallback” setting does not seem to be working. I’ve set this as our “topic list default thumbnail” but any topic that does not have an image, is not using this fallback image, as illustrated here:

Is there a solution to get the default image to show? I miss the congruence when there was an image or default image for each topic.

One other note… the thumbnails used to show animated GIF’s which we loved. But now it seems it just displays a still shot for the thumbnail, instead of the animated GIF.

One other note: we are also using the Pavilion Events Plugin, which @davidkingham mentioned they are also using.

1 Like

This works in the TC, I used it this week. As for the plugin, I’ll take a look at this in the run up of the next supported upgrade period. Thanks for the report.

Thumbnails are created by Discourse now. Most of the related logic has been removed from TLP. We would need to ask them to support animated GIFs.

Yep, I’ve raised this. This is a template clash. One of my colleagues is looking at it.

1 Like

It should work - I have seen sites using our official theme component successfully with animated gifs. Make sure that the allow animated thumbnails setting is enabled in your site settings. (it is enabled by default)

If it still doesn’t work, feel free to create a new #support topic here with the gif that doesn’t work.

3 Likes

Goooops, missed that setting, David, thanks so much for interjecting!

@jord8on if you get no joy with that I will take a look in first instance in that case. Hit me up with an example in a PM if that doesn’t work for you.

3 Likes

Animated GIF’s are working on our site. I’m not sure what the issue was. Maybe it was something on my local device ¯\_(ツ)_/¯

Thanks for looking into that!!

One more question. I just checked out the official discourse theme component and am wondering what the differences are between that Topic List Thumbnails Theme Component and this “Topic List Previews” plugin? Is one better or worse for certain situations?

2 Likes

I think it’s going to take you less than a minute to decide for yourself? :). TLP TC does what it says on the tin.

2 Likes

This is implemented in the TC, please test this out and revert:


default: OFF

https://github.com/merefield/discourse-topic-list-previews-theme/commit/fcdbdf99c0ed3b27bd0b1c1661de943c895fc101

2 Likes

@davidkingham @jord8on I’ve added the missing mobile plugin outlet to the TC that @fzngagan has kindly PR’d into Discourse core (cheers mate!), so you might find things are improved. I’ve not yet tested with the Events plugin though!

Let me know how you get on with Events plugin and the TC:

https://github.com/merefield/discourse-topic-list-previews-theme/commit/cbd1bf811a3836abebf236f5243027e4a5c29f75

If you need this in the plugin, let me know.

2 Likes

This helps us so much, because we can now publish and promote content for our members first. Thank you! :smiley:

2 Likes

If we adjust thumbnail resolution or uninstall the TLP plugin, do we need to take any actions to purge the old generated thumbnails? Or will they purge automatically in time?

1 Like

Resolution is just a superficial JavaScript level choice thing.

Good question. Thumbnail generation is a core activity. Try removing and rebaking a Topic and see if they disappear from latest.json

2 Likes

We currently see a discrepancy between the featured images order and the topic order by tag, i.e. the featured topics on https://1e9.community vs https://1e9.community/tag/titelstory?ascending=false&order=created

What exact dimension are the featured images ordered by? Thanks!

1 Like

Created At. As in when the Topic was created (not when the tag was applied). And I’ve just double checked a site I have it on and it appears to be working as expected in both modes?

2 Likes

FYI everything seems to be working great, the conflict with events/mobile does still exist though.

2 Likes

I’ll take care of this in a few hours.

p.s. I made that change. Now the TLP template should load on mobile with the event details.

2 Likes

thanks @fzngagan

So now I see an event on the topic list on mobile fine using the TC. Can you please confirm @davidkingham?

1 Like

That’s personal choice. TLP has been around longer and in part that is why it is more feature rich (Thumbnail chooser, Featured Images, User Portfolio system, support for Suggested Topics), it has a slightly different Tiles format with more meta data, TLTTC has an additional view type (grid).

1 Like