Topic: Been a while (Page 1 of 1)

I haven't tossed a site out in a while. This site isn't all that much in terms of design or anything, but I would like some opinions on it if possible. What is your impression, recomendations, etc, etc.
http://www.flrba.com

Thanks for your time,

-Dan-

First thought was that the page is really small, almost cramped. Even if you don't need more space for the content I would add some more spacing between various elements to make the page bigger. Some more space between the heading and the content, and between the menu and the content might look good.

Not much to say on the layout and design really. Nothing new there, but it is working as it is supposed to. The bevel kind of border you're using looks kind of cheap (in lack of a better word for it) to me, I would have prefered a simple black line without the shadows. But that might be just me...

You've used three <h1> tags on the first page. Sematicly I think it would be better if you used <h1> the first place, but switched to <h2> in the second and third place. If you want to keep the two other as <h1> I think you should remove the header tags (or make it <h2> ) around the logo instead of having all of them as <h1>.

There is also a little bug in Opera. Between the top header and the content/menu there is a 10px gap where the background shows through.

_________________________
"There are 10 kinds of people; those who know binary, those who don't and those who start counting at zero"
- the Golden Ratio -

I see what you mean about the space issue between the header and the content. I am unsure about changing the spacing between the content and the menu, I have to look into box model hacking as it displays differently among IE and Firebird (IE has a ton of space).

I will impiment your idea about the spacing issue tomorrow when I next work on it. Thanks for the ideas.

-Dan-

WareMage, I found a fix for your opera bug.

#head {

	background: url("h.gif") no-repeat;

	width: 620px;

	height: 125px;

	margin: 20px auto 14px auto;

}

#head span{

	display: none;

}



#page{

	width: 620px;

	background: url("b.gif") repeat-y;

	margin:-14px auto 0 auto;

	padding:0;

}

I changed the bottom margin on the header to 14px, as that change doesn't get picked up in Opera and levels the playing field. Once you have that 14px space in all browsers, then you simply give the #page element a -14px top margin.

All in all, I think the site does it's job. The size doesn't bother me because it fits the small amount of content. I never really cared too much for shaded borders, but that's more personal preference.

Hope this helps. BTW, your CSS file is very clean and simple, nice work.

EDIT: While I was fiddling, I noticed the site was not centered in IE5, just add a container div around everything with a text-align:center in the CSS and all is good. On second thought, you then have to add a text-align:left; to your content div so all your text is not centered.

One other quirk, there are no styles in the table on the contact page in IE5. I guess it doesn't pick up the body text declarations.

Ramasax

[This message has been edited by Ramasax (edited 01-17-2004).]

Thanks for the look and the fix.

-Dan-