[TH] Nodes for grid, custom styling, and custom icons [Deleted]

Thank you, I was wondering if this had been changed but I was able to reproduce the error yesterday with the same value. Is on my list to fix.

We have also same issue with PHP 8.
ErrorException: Template error: [E_WARNING] A non-numeric value encountered internal_data/code_cache/templates/l1/s11/public/th_node_list_grid_nodes.less.php:99
 
Bug fix should be out in hot fix
We can probably agree to disagree on that account. A major php version compatibility update is not something that is commonly provided in a hotfix. XenForo has taken multiple updates to provide full compatibility, none of them was a hotfix. MediaWiki is still not compatible with php8 to this date after three major releases. These examples are obviously a magnitude larger in code base, but my point is regardless that this is not thrown out in a hotfix.

Any cPanel server host will allow you to switch your php version from 8 to 7.4 without any service interruption too, and switching on a self-hosted server shouldn't be an issue either, so I really don't see a reason for the rush. We'll update it when we update it, there hasn't been a promised timeline.
 
Just wanted to make sure - does this addon allow to set everything separately on each style (both colors and background graphics)?
 
I have just start to work to increase my score at "https://gtmetrix.com/".

How can I add lazy-loading feature to TH nodes image files ?

The simplest way would be to add "loading=lazy" to the <img> tag. This will likely require a template edit. For example:

HTML:
<img src="image.jpg" alt="This Is My Image" width="500" height="250" loading="lazy">

You can see a significant performance gain on pages heavy with images "below the fold" (viewable only after scrolling). In turn Google's Pagespeed, GTMetrix, Pingdom, Webpagetest, etc. all respond well to this method.

This browser level lazy loading is supported by most popular Chromium-powered browsers (Chrome, Edge, Opera) and Firefox. No Safari support at this time.
 
I think youd need a lazy load script of some kind. If you load it in, it should lazy load everything not just your node images. Avatars, logos, thumbnails, all kinds of images I assume it would cover. Nothing to do with this product though.
Thanks for your answer. As far as I know XF 2.2 already has "lazy load" feature as @bzcomputers mentioned at his post.

But I can not use XF Lazy feature with this addon due to its a bit complex and does not use <img> tags.
The simplest way would be to add "loading=lazy" to the <img> tag. This will likely require a template edit. For example:
Thanks for answer. I know this feature and already using at every possible situation like widgets with background pictures....

The problem for this TH NODE, it is very special and does not use <img> tags.

As far as I know this addon uses "th_nodeStyling_nodes.X.less" templates to create nodes pictures. And there is a warning at these less files

Do not edit this template directly, everything contained within this template is automatically generated and manual changes will cause issues

Screenshot_1.webp


Here is an example how it creates pictures for a single node.

Code:
.node--id1 > .node-body {
        background-image: url(https://www.rckolik.com/forum/styles/rckolik/nodes/duyurular1.jpg) !important;
        background-size: cover;
    }

Screenshot_2.webp
You can see a significant performance gain on pages heavy with images "below the fold" (viewable only after scrolling). In turn Google's Pagespeed, GTMetrix, Pingdom, Webpagetest, etc. all respond well to this method.
Yes, totally agree with you (y)
 
Back
Top Bottom