|
| |
|
|
phoon
Posts: 3
Joined: 4/8/2008
Status: offline
 |
Please take a look at my web site!! - 4/8/2008 23:10:03
Hello,
Please take a look at my site. I am currently a student of Internet Application Programmer student. This critique request is part of my eCommerce course requirement.
Some of the pages, links or buttons of my site are still not complete. They will have alert box or "Under construction" as a message.
Thank you in advance,
Phoon
http://acadunix.algonquincollege.com/~cier0003/
|
|
|
|
Ryokotsusai
Posts: 248
Joined: 10/5/2005
Status: offline
|
RE: Please take a look at my web site!! - 4/9/2008 0:49:54
Hello :D
It seems empty, not a lot of color, or really much of anything else, kind of reminds me of an older site (minus the animated gifs thankfully), if 'Ken's Butchery' is a real store, then one of their cards or ads may be a good place to get an idea for colors for the site, as the gray on gray on gray is a little dull. The lack of any changes to the default link style adds to that as well.
Also the banner is really pixelated, using the width / height properties to resize an image is rarely a good idea; there are quite a few decent free image resizing tools out there that can handle that.
Another thing is, you may want to bring the width of the containing div down to around 750px so visitors with 800x600 screens don't have to scroll side to side.
< Message edited by Ryokotsusai -- 4/9/2008 2:40:48 >
_____________________________
The world is more like it is now than it ever has been before.
--Dwight Eisenhower
| |
|
|
|
womble
Posts: 5445
Joined: 3/14/2005
From: Living on the edge
Status: offline
|
RE: Please take a look at my web site!! - 4/9/2008 5:10:40
I'd agree with Ryokotsusai's comments.
Also, as it's a site for a course, you've got 3 errors in your HTML. That might not sound like a lot (and it's a hell of a lot less errors than we find on most sites posted for critique here), but having done courses myself where I've had to produce pages/sites for assignments, you usually lose some marks if your pages don't validate to the chosen doctype. The errors you've got in there on the home page are simple ones, and easy to fix, just make sure you run all your pages through the W3C validator, but all can help you pick up those valuable marks.
Make sure as well that you're using heading tags properly (<h1>, <h2> etc.). On your project pages, for example on the details page of the "All Beef Sausage" page, you have "All Beef Sausage" inside a <p> tag surrounded by a <span> tag. Span tags are okay for applying styling to selected bits of text, but as "All Beef Sausage" is a heading, it should really use a heading tag, and then apply your styling to the heading. On that page you have "Ingredients" and "Nutritional Information" as headings, but not the "All Beef Sausage". It might not make any difference to how your page looks visually, but for your pages to make sense semantically, for example for a screenreader or search engines (Search engines pay particular attention to headings), you should be using heading tags appropriately and hierarchically. Similarly on that page you have your "Return to Order Page" link as a <h4> heading. As an internal link, that doesn't need a heading tag and is fine as just an ordinary <p> tag.
As Ryokotsusai says, the look of the site is very 1990s, and though I know often the briefs for assignments aren't particularly inspiring (I've got an assignment due in on a course in a couple of weeks that includes a question on producing a web page and the instructions are that it must include text, an image and a hyperlink *yawn*) but try looking around online for examples of similar sites to get an idea of the sorts of colour schemes and graphics used on similar sites. Some assignments like these have marks allocated for styling and some don't, and I'm not suggesting you spend huge amounts of time on the styling of the site, but showing your tutor you've taken the time to consider such things is always good, and is a good habit to get into for later on when you're developing your own sites.
_____________________________
~~ "A cruel god ain't no god at all" ~~
| |
|
 New Messages |
 No New Messages |
 Hot Topic w/ New Messages |
 Hot Topic w/o New Messages |
 Locked w/ New Messages |
 Locked w/o New Messages |
|
Post New Thread
Reply to Message
Post New Poll
Submit Vote
Delete My Own Post
Delete My Own Thread
Rate Posts
|
|
|
Printable Version 
