Small Changes

[fisicx]

Had some good feedback in other threads about my theme most of which have been implemented.

One thing you can do is show featured images at the top of each post or page. Sometimes this can be a bit too much but if disabled can leave he page a bit bare.

Whilst I'm not a fan of hero images, a small banner image can work well.

So that's what you can do.

https://tradesman-theme.co.uk/

Click on any of the links and you will see these images. They are random (you can have up to 8 images) and if you hover you can see the image caption.

Does it work for you visually? Do you even like it? Will England ever win a World Cup?

 

[Nico Albrecht]

Looks good but you might wanna check the contact page: https://tradesman-theme.co.uk/contact/

Notice: Undefined index: coming up several times.

 

[fisicx]

Sorted. That's what happens when you fiddle with things.

 

[dotcomdude]

Does it work for you visually? Do you even like it?

This page: https://tradesman-theme.co.uk/download-and-install/

Shows this error:

Warning: Use of undefined constant header_image_post - assumed 'header_image_post' (this will throw an Error in a future version of PHP) in /home/tradesmanthemeco/public_html/wp-content/themes/tradesman/single.php on line 3

Need to look at your error reporting setting I guess...

 

[Simon.P]

Looks great @fisicx i will make use of this feature. Do i need to upload the whole theme again, or is it just the functions.php and css file?
Thanks for a great theme

 

[fisicx]

Should be able to just install the update. I'll send it to you when I've finished today's upgrade - adding a schema option.

 

[Simon.P]

That would be amazing, thanks @fisicx

Edit: I forgot to say i made a child theme (because i have made some tweaks like using the nav bar from your take-the-lead layout). It works but there is an error (link below) so not sure if this will cause any more problems?

Thanks

https://drive.google.com/file/d/19VRzWchqufweR9uRQVlBV-IOHVF7A6bL/view?usp=sharing

 

[fisicx]

All done.

https://tradesman-theme.co.uk/download-and-install/

Just need to overwrite the existing theme

No idea why you see that error. But I will investigate

 

[Simon.P]

Thanks. Updated. The child theme is no longer recognized. I've tried deleting and re-loading the /tradesman-child folder but no joy. Any ideas?

 

[fisicx]

I've just created a child theme without any problems.

https://tradesman-theme.co.uk/wp-content/uploads/tradesman-child.zip

 

[Simon.P]

Thanks @fisicx all working OK. And the error has gone about deleting the missing theme.
Cheers

 

[Simon.P]

Hi.
Any idea how i can remove the zoom function on images. On a page like this for example:

https://www.simonportillo.co.uk/product/rcd-video-call-help/

 

[fisicx]

That's a woocommerce thing.

A solution given here: https://generatepress.com/forums/topic/remove-zoom-effect-on-woocommerce-product-image/

 

[Simon.P]

Thanks for all your help @fisicx , that worked perfectly.