Onderwerplijstvoorbeelden (TLP)

@merefield - i want to apologize for the inconvenience for not testing thoroughly enough - the trouble came from the unmaintened TC Topic Banners component - i did not realize that it was set to unmaintained some time ago. and it interfered somehow with your TC.

do you think i should suggest a new discourse feature to show the TC status in the respective admin panel?

1 like

No problem. I think the core team has already made some efforts to improve that situation. I’m not sure if there is more they can do but in any case don’t worry. Sometimes it’s hard to tell where the issue is coming from :).

I seem to have a problem that once I infinity load the topic list - the layout is broken - to verify simply open this URL - use theme toggle (left bottom) and switch to retro and scroll all topics until you find the problem: https://neidan.discourse.group

Screen:

As you can see in the screen the layout is broken after the infinity load… the thumbs and all is mashed together…

The forum is officially hosted by discourse so there is a very low amount of plugins and modification.

I cannot recall having this obvious problem before - it must have happened with an update…

1 like

Curious. And no errors in the console. I will take a look when I get chance. Thanks for the report. (nice looking site btw :slight_smile: )

(FYI The lack of native Masonry rendering in the browser is an annoying omission that forces us to write more fragile Javascript. This Theme Component is over 8 years old now and in all that time they are still arguing over the implementation!)

I can repro. Looks like this broke https://starzen.space too after an update.

Try that for size (I think it may be related to the size calculation when off-screen - a workaround existed but was restricted to Firefox)

Please update and let me know if it resolves your issue.

Hello! Thanks for the quick fix! It seems to work - it seems like a workaround though because the first millisecond I see the same problem but then it readjusts itself. However the results work for now. thank you!

1 like

Oh it’s more than a “workaround” in that sense. The JavaScript is designed not to impose too much. If it ran more often it would waste energy. It’s a compromise until masonry is supported by CSS …

I see! thanks a lot for the quick fix! :slight_smile: I will stay with your component and hope it will happen soon.

Don’t hold your breath haha

2 likes

one last thing I noticed the mobile view on my iPhone 13mini is a little more off than usual - here a screenshot

this is definitely worse than before. any way to fix this? Text and Image go way beyond the border.

@patrickemin you could try this css too:

.tiles-style .topic-list-item.tiles-style  {
    background-color: var(--primary-very-low);
}

to give you a simple background colour to help differentiate the tiles.

then you would not need the sidecar plugin.

For sure it can be fixed with CSS. If you need help Marketplace

The Component is open source and there is no subscription, but I’m afraid my free support is limited as I’m a busy guy.

I’m not seeing that issue on either of the sites listed in the OP? (feel free to confirm on your device) Suspect it may relate to the special requirements of your specific theme.

Thanks, but all tiles are affected ? Ideally only one tile on two (even tiles) would be with this background, but I guess this is not possible with CSS ?

I would recommend hiring someone if you have special needs Marketplace

1 like

I’ve added another fix that should make things work much better:

This should resize the tiles when the number of topics changes.

I’ve got hit by a corner case lol

I want to have 360p animated avifs for topic previews, so I set:

It worked fine until I opened a topic and scrolled down to recommendations :joy:

Apparently recs don’t get size limited for some reason
(and image and card link to top of topic and not to the new post (like title link does) but that’s another problem)

2 likes

Een suggested toegevoegd aan Topic list tiles topic lists, het lijkt erop dat het enigszins is opgelost.
Nu proberen het goed op te lossen.

1 like

@merefield
Kunt u alstublieft controleren of de afbeeldings- en detailkoppelingen linken naar /t/foo/123 in plaats van t/foo/123/1, zoals de titellink doet?
Ik denk dat het ergens een bug is
Het breekt ongelezen onderwerpkoppelingen (ze zouden duidelijk naar het nieuwe bericht moeten linken)

De afbeelding is bedoeld om te linken naar de OP, aangezien daar meestal de miniatuurweergave staat. We zouden instellingen kunnen toevoegen om meer opties te bieden, waaronder (maar niet beperkt tot):

  • laatst gelezen bericht en
  • momenteel geselecteerde miniatuurweergave (die niet in het eerste bericht hoeft te staan, aangezien je er een andere kunt selecteren)
3 likes