Topic List Previews Theme Component

Oh there is no urgency. :+1: Please check it out when you have time according to the priority. Meanwhile I’ll test some more and report if there is any issue.

1 Like

BTW am seeing overlaps of topic list items on category pages with display of sub-categories, this might affect user experience…

That’s probably down to a JS error from either the TLP TC or something else.

The layout requires working JS. If something breaks the JS on the page, it can halt the TLP “renderer”.

What happens when you refresh the page?

Do you see an error in the console? I cannot currently repro, e.g.: Uncategorized - StarZen: The Unofficial Star Citizen Community

I successfully reproduced it (still tiles and excerpts enabled).

And yes, when refreshed the layout became normal, for a short period. Btw the blank at the bottom of tiles doesn’t seem normal. I didn’t see any errors in the console other than a “add a pluginId key to your modifyClass call” reminder.

After refreshing a few more times, the layout broke again in the way shown in the first image.

Hmm the category in your link doesn’t have a sub-category, it seems. The layout break only happens when there’s a sub-category list on the category page.

Cannot repro that either, just created a sub category, navigated to it and tiles render fine.

Try removing all other TCs and plugins and work out if something is causing a break.

1 Like

Hey Robert

found another small issue: Whenever TLP tc is activated, even without enabling it in a specific view, the width of the div tag with the class category-and-tags is squeezed together in regular topic list. I could not pinpoint the exact CSS which causes the issue though.

With TLP enabled

Without TLP enabled

I made this change in response to your prior feedback
This is deliberate to accommodate actions. It’s a trade off. When we have container queries on all main browsers I can make this more intelligent. Firefox is currently outstanding.

It definitely looks more odd because you haven’t enabled actions.

Here’s an example with them enabled:

If you are fine with the end of tags not lining up vertically with actions one could design this differently but that would be wasteful in an different way

I might be able to add a tweak to reduce the actions ‘column’ it they don’t exist.

Tbh tiles is the most sophisticated layout option for mobile

Hey merefield. Thx for the detailed explanation.

In my specific case I have TLP enabled solely to display the topic thumbnail picker (while all other settings are disabled).

if there was a way to display the thumbnail picker without TLP tc active, I would go with that. Hence I submitted that "bug"report. Since the TLP TC is active without having any active settings, I did not expect

If this “bug” is not critical, then I can submit a feature request for an option to activate the thumbnail picker via plugin options without the need to have TLP TC enabled. Is that something feasible?

I’ve tweaked it to give you (quite a bit) more room if you aren’t using actions:

Please update and give that a whirl

3 Likes

Hey Robert

thx for the quick fix, realy appreciate it! It’s already closer to expected behaviour, but since the space on mobile is so highly limited, we would like to maximize it.

Following up on this. Such a feature would eliminate the issue above.

The TC and the plugin are only split so that some users without plugin capability can make use of the basic features and provide very fast rollouts of front end changes.

I have no plans to change that division of responsibility at present and plan to keep all front end code in the TC. My time is also pretty limited.

1 Like

I broke the Topic List Like in a recent refactor, apologies. This is now fixed FIX: refactor broke Like · paviliondev/discourse-tc-topic-list-previews@f41d9de · GitHub

1 Like