headerWrapper too big

[LizzyB]

Hi,

I am having a problem with a gap beneath my logoWrapper image. I know a bit about CSS and have been using Firefix to try and debug the problem (using view CSS) I have narrowed doen the problem i think that the #headerWrapper appears to go below the logo - for testing purposes I have coloured it bright green in the stylesheet. It has no padding/margin so i dont see why it is showing. I want the image to ######## straight against the main content.

Please view my store here:
http://westmontglass.co.uk/zencart

Thanks
Liz

[schoolboy]

Try this:--

#headerWrapper{
    background-color:#00CC33;
        height: 200px;
    } 

[LizzyB]

Thanks for the reply. Trouble is that #headerWrapper emposases more than just my logo (I have found the code for it in /common/tpl_header.php) .If you set the height to the same as the logo is then wipes out the top bar with my log in etc.

Really #headerWrapper is just a holder and as it has padding and margin set to 0 in the CSS


#headerWrapper, #contentMainWrapper, #logoWrapper, #cartBoxListWrapper, #ezPageBoxList, #cartBoxListWrapper ul, #ezPageBoxList ul, #mainWrapper, #popupAdditionalImage, #popupImage {
margin: 0em;
padding: 0em;
}


i cant see why on earth its giving itself extra space.....

[LizzyB]

OK I think i have found whats causing it but not sure how to get round it.

In common/tpl_header.php there is a <br clear=all> line 92 which i suspect is causing the extra space - (well i tried removing it and it fixes the problem)

BUT my problem is that by removing it i may cause a problem down the line if I change my mind about what i display and dont display in the header section.

Actually inspecting the code further I wonder if the clear should be moved before the closing div as on line 65/66

so line 95/96 should be changed from

</div>
<br class="clearBoth" />

to
<br class="clearBoth" />
</div>


Any advice welcomed!

Thanks