Jump to content

[SOLVED] Please evaluate


rhodrykorb

Recommended Posts

First impression, very nice!

Quickly searching trough the source for h1/h2/h3 also very nice.

 

Closer look at the code, not so nice.

<table summary="layout" id="layout-table">

And that's where things go wrong, table != layout

Table = Tabular data

Div    = Layout

 

Form on the right is using labels for the form, good job.

Thought it was a bit slow, and yslow is showing 7!! external javascripts. Hope you plan on combining those where possible.

 

Not much else to say as there isn't any content yet.

 

Looks pretty sharp.  Nice use of graphics and good spacing.  I'd consider increasing the line-height slightly for the text.  It looks a little cramped. I'd also prefer the text be aligned with the logo/text at the top to make the grid look good.  I'd also add a little top padding to the teachers/students/administrators section at the bottom.

 

Axeia is right that you shouldn't use tables for design purposes, but it is not the end of the world. 

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.