|
Posted by Chaddy2222 on 12/11/92 11:49
Rik wrote:
> Chaddy2222 wrote:
> > Hi all.
> > I am currently re-designing my site to use a CSS based layout. I was
> > just wandering if anyone could give me some feedback on how it looks,
> > suggestions on how it could be improved that sort of thing.
> > The test page is at: http://freewebdesign.cjb.cc/test
> > Only the index page is up at this stage, I will add content and build
> > the site from the index page once it's working well.
>
> <meta http-equiv="Content-Type" content="application/xhtml+xml;
> charset=ISO-8859-1" />
>
> Hmmmmz, not sure wether the choice "application/xhtml+xml" was appropriate,
> but I've forgotten the exact rules :-).
Well, it can be done, but a lot of people say that HTML4.01 strict
works just as well.
It's actually served as HTML but it has the XHTML info their. If IE had
better support for XHTML then the pages could be served up as XHTML
instead of HTML, but XHTML is not well supported yet.
>
> <div id="header"><h1>Free Web Design Online!</h1></div>
>
> Why wrap it in a div? What layout are you giving it, that you cannot apply
> to the h1 directly? Div's are nice, but terribly overused. If there's a
> better tag, use it, maybe using a couple of different classes.
>
> Navigation: it's actually a list, you might consider making it one,
> allthough removing the standard browser presentation is a pain. You seems to
> have started out like that, considering the seemingly useless:
> list-style-type's in the "a" markup.
Yeah, I removed the list from the navigation as I could not get the
page to validate with the list, now i've got it validating I will add
it back in.
>
> From a graphical point of view:
>
> H1: try some padding on the left, I think that will look a lot better, but
> that's personal taste.
I'll have a look in to that, thanks.
>
> The "navigation" at the top is not enough "seperated", in Opera 8.52 I can
> hardly make out that it's not just one link. Also: try to be consistent in
> the use of capital letters. Now it's mostly random.
>
> Text color is OK to my young eyes, but I'd imagine that for some people i
> could use a bit more contrast.
>
> I'd suggest using this in the body:
> <body>
> <h1> Free Web Design Online!</h1>
> <ul>
> <li><a href="#" title="Home" class="on">Home</a></li>
> <li><a href="index.html">About Us </a></li>
> <li><a href="index.html">Web Design </a></li>
> <li><a href="index.html">Contact us </a></li>
> </ul>
> <h2>Welcome To Free Web Design Online!</h2>
<snip>
Thanks for the list suggestion.
> And adjust your css accordingly.
> About using lists: http://css.maxdesign.com.au/listamatic/ contains a lot of
> examples of how to whip them into the shape you want.
>
> I applaud the use of relative sizes for text instead of px (or even worse,
> pt..).
>
Thanks. My current site at http://freewebdesign.cjb.cc can also be used
with user specific font sizes, however I was having trouble controling
issues such as keeping the font sizeing constant across all pages with
the old layout.
--
Regards Chad. http://freewebdesign.cjb.cc
[Back to original message]
|