Hi,
Add this two lines of code:
$buffy .='<a itemprop="url" href="' . home_url() . '">';
…
$buffy .='</a>';
..before and after the logo img tag, like this: http://screencast.com/t/IFfDpBTxm
Thanks
Thanks
I’ve tried it but nothing changed so far. Here’s my php file (it’s been uploaded via FTP to the server), and the firebug inspector:
http://www.screencast.com/t/Lbgxw47Fe3wq
Just one more question: Does this mean that after another Newspaper update I will have to change this all over again? I had the same thing with integrating social icons on the front page. I updated the theme yesterday and all my code was overwritten by the new version.
Hi,
I also tested this and Lucian’s solution works: http://screencast.com/t/frGguJJfNkS
But I saw your screenshot and it looks that you added the code right.
Regarding your question, if you have customization/changes directly into to the theme’s files, these files will be overwritten. If you do have modification to the code you should make a copy of the files changed by you, updated the theme and make those changes again if the files where changed.
So, if you updated the theme after that you added the code for footer logo is possible that the file was overwritten.
Thanks!
Hmm… I’ll keep trying to find what is wrong with my implementation of Lucian’s code. I’ll post back in 2-3 days. Please be patient till then.
Hey there
I think I figured it out but I am not sure if I’m correct.
It seems there is some issue with the regular logo vs retina logo. If I change line 75 from:
if(!empty($instance['logo_url_r'])) {
to:
if(!empty($instance['logo_url'])) {
then the footer logo image starts working as a link.
Any idea how to get the theme to disregard the difference between retina and non-retina image? Or maybe you can tell me where/how to upload the logo file to make it work without changing line 75?
Thanks
Sorry for another post, but I think it works – I uploaded the retina version to the same folder and put the URL to the new retina logo in the second field (the one for the retina version). With both logos, everything started working.
The lesson here might be that not everyone uses the Retina version, so maybe you could take it into account in future updates.
Consider this topic closed
