Site logo

Archived topic

Google speed index: specify height and width of image

25 replies · Started by Henk on January 13, 2022

Viewing posts 16–26 of 26

to test the responsive mode with the new code

Seems the code is not working, can you make sure?

Hi Yin,

I double checked the functions.php.
See private area for FTP-login. No problem, to have a look and/or change something.

I'm not able to log in with the logo info.

Can you check?

Hi Yin,

Mmm, may be I made a copy-past error, please see the private area again for the credentials of SFTP and SSH.
I double checked the credentials.

I forgot to mention it's SFTP, not FTP, may be that's the issue?
And kinsta also requires a specific port to connect to.

If you check functions.php, it's good to know that the child theme Simmelink is used.

Hopefully, you can find the issue.

Can you provide the Wordpress login?

Sure, see the private area.
Feel free to make changes or try things out.

Please let me know if you need something.

I think the issue is from the merge function of the header element.

It generates a mobile page hero logo which doesn't have height and width.

It's a potential bug, we'll look into it and try to fix it in the next update.

For now, I set the header element tomerge on desktop only, add this CSS to customizer > additional CSS to manually merge the mobile header:

@media (max-width: 768px) {
    .header-wrap ~.page-hero {
        margin-top: -70px;
    }
}
.header-wrap #site-navigation:not(.toggled) .main-nav > ul > li > a, .header-wrap #mobile-header:not(.toggled):not(.navigation-stick) .main-nav > ul > li > a, .header-wrap .main-navigation:not(.toggled):not(.navigation-stick) .menu-toggle, .header-wrap .main-navigation:not(.toggled):not(.navigation-stick) .menu-toggle:hover {
    color: white;
}

Once the bug is fixed, you won't need this CSS anymore.

Hi Yin,

Thanks for your help and time to find a solution, I really appreciate it.
Am I right that you already changed the other Elements that I use for some pages?

And what should I do with Autoptimize? Just enable it again?

Am I right that you already changed the other Elements that I use for some pages?

Yes, I switched the mergeoption to merge on desktop only for other header elements as well.

And what should I do with Autoptimize? Just enable it again?

Yes, correct.

For the question in private info field, I'm not familiar with those plugins Unfortuanly. I can't give you professional advices, sorry for that.

Thanks for all your work.

You are welcome :)

This archived topic is closed to new replies.