Space - Needs decreasing

In PNL-CSS-2-Column.css  change:

.PNLBody #sidebar1 {
      float: left; /* since this element is floated, a width must be given */
      width: 150px; /* the actual width of this div, in standards-compliant browsers, or standards mode in Internet Explorer will include the padding and border in addition to the width */
      background: #993300; /* the background color will be displayed for the length of the content in the column, but no further */
      padding: 15px 10px 15px 20px;
}

to reduce the top padding. that is where the extra height is coming from.

Cd&

Cd&

Thanks that raises the Sidebar and makes things neater but, there is no padding on the #AdSpaceTOP (apart for the px on the inside of it) How do I do the same with that?

IanTh
>>you have a container div thats meant to go around the whole screen
>>you need to start using divs if you want instead of tables

Agreed I'm getting there slowly :)

Pete,

You can try a negative margin, but be careful with that because it might create issues for old IE versions.

I didn't even realize there was an ad there.  My adblocker is very effective.

Cd&

>> My adblocker is very effective.
Ah I See

I've looked at both those elements see att screenshots - There seems to be no reason for that spacing at all between the headerDiv and the AdSpaceTOP Div - the problem repeats itself below, with the same amount of space above the mainContent Div and below the AdSpaceTOP Div.

(I'm very weak on this subject - so its probably something obvious).

Pete
Ad-Div.jpg
Header-Div.jpg

I am not seeing anything out of order with your CSS coding here. Your margins and padding attributes are properly set for what you are attempting to do. However, while examining your live code I noticed that the #header section is also entirely enclosed within an <h1> tag with no related textual content. This could be causing a spacing issue due to a default linespace attribute.

The code looks like this when collapsed:
<div id="header">
  <h1>
..... <!-- Header content -->
  </h1>
</div>

wadehults

Hi Thanks, that's because the CSS was originally fromone of Dreamweaver's built in templates, I'm guessing it assumes that content in that div would be <h1>

Anyway, I agree its not needed and I've removed it, and still the gap remains!

Hi CD&

Sorry Dropped of the net for a couple of days, I wasn't sure what you meant, but did some searching and came to the conclusion that you meant add...

p.inline
{
display:inline;
}

to my CSS? (please confirm). I've done so, nothing has moved, so I've probably not understood?

Pete

-----Ah, I removed the <p> Tag completely form the AdspaceTOP div, (It was only there to center the ad in the Div), I added the allign="center" to the Div instead. And everything jumped to where it should be! ( I need to add a few padding pixels to line things up but that wont take me long).

Thanks to everyone who helped me out, especially Cd&, who hit the nail on the head!


Pete

you dont need to worry about old ie versions as its now being automatically updated to the latest so just make sure your code works on the latest version