XF 1.4 Lose this annoying gap

[JamesBrown]

Can anyone show me ho w to lose this gap, so my logo isn't floating? Thanks www.motorhomefun.co.uk

 

[rainmotorsports]

Not sure where its floating:


The actual logo has a pixel or two top and bottom?


Perhaps there is something going on in your screen that I am not experiencing or maybe i misunderstand?

Oh wait disabled ad blocker. I see it. Lets see. Also thanks for adding the image.

 

[Paul B]

Remove or reduce the margin top from the ad.
Set it to 6px instead of 17px if you want it vertically centred.

 

[rainmotorsports]

Remove or reduce the margin top from the ad.
Set it to 6px instead of 17px if you want it vertically centred.

They may be trying to prevent the login tab from covering the ad. Not sure if thats in the rules or not.

If the ad's are fixed in size and they don't mind the top margin when the ads are not present... could add it to the logo instead. I know another solution that unfortunately doesn't work with the way the theme is built.

 

[Paul B]

Good point.
Having a click element over an ad is against Google's TOS.

It will require a different solution.
EXTRA.css:

#logo img
{
    margin-top: 11px;
}

 

[JamesBrown]

It will require a different solution.
EXTRA.css:

#logo img
{
    margin-top: 11px;
}

Thanks Brogan. It looks much better now

 

[Russ]

Thanks Brogan. It looks much better now

Not quite, logged out is messed up now.

 

[Paul B]

I did it logged out.
What are you seeing Russ?

Edit: I presume you're browsing with a small screen? Yes, you'll need to remove margin top using a media query for when it's less than max width.

 

[Russ]

Nope:

 

[Paul B]

Ahh, you have ads blocked.

Personally I'd ditch the ad in the header, as that is what's causing the issues.
Either that or make the logo the same height as the ad.

 

[Russ]

Looks like removing the margin-top on the logo, and setting the margin top on the ad: 6px will make it work with or without the ads.

 

[Paul B]

setting the margin top on the ad: 6px will make it work with or without the ads.

The ad overlays the login handle then.

https://xenforo.com/community/threads/lose-this-annoying-gap.83424/#post-829121

 

[JamesBrown]

Looks like removing the margin-top on the logo, and setting the margin top on the ad: 6px will make it work with or without the ads.

Thanks Russ and Brogan. Where will I find that margin setting on the logo and the top of the ad?. Cheers

 

[JamesBrown]

Sorted (I think)

Style Properties: Header and Navigation > Height of Header Logo Added 5px

Thanks again.