Topic: Site Review Again Please : LHA (Page 1 of 1) Pages that link to <a href="https://ozoneasylum.com/backlink?for=10324" title="Pages that link to Topic: Site Review Again Please : LHA (Page 1 of 1)" rel="nofollow" >Topic: Site Review Again Please : LHA <span class="small">(Page 1 of 1)</span>\

 
Nethermind
Bipolar (III) Inmate

From: under the Milky Way tonight
Insane since: Feb 2003

posted posted 03-12-2003 05:29

This is an updated link for a site I have begun work on, my first using extensive css as it was meant to be used. I think I actually got through the whole page without using a single table (ack!). This is only the first draft of the new version, so don't expect any links to work or any content. This is a design review request.

Louisiana Housing Association

All polite ripping me to shreds gratefully received. Thanks!


krets
Paranoid (IV) Mad Scientist

From: KC, KS
Insane since: Nov 2002

posted posted 03-12-2003 05:33

Lots and lots of white. Seems like you could use something in the upper right corner of the screen to occupy all that empty white space. The rest of the layout seems fine to me but that big empty space is distracting; my eyes keep wandering up to it.

Technically speaking your boxes on the right and left are floating over the main body text instead of causing it to wrap.

Other than those two things I like it. Nice and simple and easy on the eyes.

:::krets.net:::

DL-44
Maniac (V) Inmate

From: under the bed
Insane since: Feb 2000

posted posted 03-12-2003 18:43

Much much mcuh better.

Krets has a good point about the whitespace in the top-right. There are tons of options for that.

The layout has a bit more to it this time around, and has a bit more of a logical flow.

One note on the code side - on the left side, where you have the 'Information 1' etc: Rather than use <p class="title">, use an <h2>, as that's really what they are. The <p> tag should really contain a paragraph, not text styled as a header.

You can set the style for the <h2> the same as you have for title, or you can simply call the <h2> as <h2 class="title>.

All in all, a vast improvement - good work.

Nethermind
Bipolar (III) Inmate

From: under the Milky Way tonight
Insane since: Feb 2003

posted posted 03-12-2003 18:54

Thanks, guys, for very constructive criticism. I will change the <p class="title"> tags to H2s, DL. It's logical to use Header tags - I always forget to use them.

krets - I got the template for that css design from glish. Is there a way to have them wrap instead of overlay the center? I put the top margin at auto because it was screwing up in lower resolutions.

Now I just have to think about what to do with the empty white space on the right.


~N



Post Reply
 
Your User Name:
Your Password:
Login Options:
 
Your Text:
Loading...
Options:


« BackwardsOnwards »

Show Forum Drop Down Menu