Adrienne Boswell said:
I volunteerd to do the PTA web site for my son's elementary school, and
need some feedback from the community. The url is
[
http://jmpta.adrienneboswell.com].
All suggestions, comments, bug reports are most welcome. When responding,
especially with a bug, please indicate browser and operating system.
Hi Adrienne, just a couple of remarks. First, nice, as to be expected.
* I like the yellow with the dark blue. And plus, it is useful to
co-ordinate with the needed dark bg of the nav links and white/yellow
text on these and the right sidebar.
* I would lose the borders on the sidebar. First it looks better
without, Second, you are already fighting for width space (horiz bars
loom early for users with thin browsers and you are guarding against
early float drop.)
* Suggest in the fight against width, instead of 1em all around,
#content {padding: .5em 0 .5em .5em; ...}. You already have white buffer
to left of the container element without any padding (put a thin border
on container to see)
* Suggest to add #header address { ... padding-left: .5em; ...}
And now some other ideas, fancy things maybe, but if they work, would
make it look sort of neater without the yellow hanging at a different
height to the blue.
If you can be reasonably confident of the nav left not being so full of
content that it would push the yellow below the content:
One more wrapper! And a very few additions to CSS.
<div id="bigWrap">
<div id="nav"></div>
<div id="container">
<div id="content"></div>
<div id="sidebar"></div>
</div>
</div>
<div id="footer"></div>
CSS:
#bigWrap {overflow: hidden; background: #ffd700; ...}
#container {background: #fff; ...}
#content {padding: .5em 0 .5em .5em;...}
and
take border off nav
You likely don't need a min-height on nav now...
Enough for now maybe.