Jump to content


Please Critique ShortkidDesign


  • This topic is locked This topic is locked
4 replies to this topic

#1 shortkid422

    Young Padawan

  • Members
  • Pip
  • 8 posts

Posted 13 June 2006 - 05:01 PM

I finally finished the site after putting it off for a descent amount of time. Link is below:

http://shortkid422.com/

Any comments/suggestions/flaming(not really)/questions/concerns, etc... are greatly appreciated. Thanks!

*Note: images within the text may be added shortly depending on upcoming work and travel.

#2 c-c-C-C-UNIT

    Pizza Hut Boy

  • Members
  • Pip
  • 240 posts
  • Location:Blue Hill, ME

Posted 13 June 2006 - 06:21 PM

You should have an active state for the navigation so users know where they are.

Also the text is much easier to read if you make it bigger one notch.

You also failed XTHML validation on two of the pages (23 errors on the contact page - http://validator.w3.org/check?uri=http%3A%...%2Fcontact.php) and one on another (forget which) http://validator.w3.org/check?uri=http%3A%...com%2Fabout.php

Edited by c-c-C-C-UNIT, 13 June 2006 - 06:26 PM.


#3 Matheus

    P2L Jedi Master

  • Members
  • PipPipPipPip
  • 1,058 posts
  • Gender:Not Telling

Posted 13 June 2006 - 06:32 PM

good website
well coded, no graphic problems
If I were you, I would take the reflection of the text and make it more translucid (transparency about 30%), to make it look like a real reflection

#4 Wolfe

    Young Padawan

  • Members
  • Pip
  • 102 posts
  • Gender:Male
  • Location:Louisiana

Posted 13 June 2006 - 07:05 PM

Simple, sleek.

I likey.

Yea, I think the reflection would look better if you lowered the opacity, too.

Edited by Wolfe, 13 June 2006 - 07:06 PM.


#5 shortkid422

    Young Padawan

  • Members
  • Pip
  • 8 posts

Posted 13 June 2006 - 07:29 PM

Thanks a million c-c-C-C-UNIT! It was getting invalidated because PHP added a hidden form field to pass the PHPSESSID, but puts it directly after the <form> field, invalidating it because it isn't contained (putting the entire form in a <div> doesn't work).

For anyone with this obscure problem, put this in the beginning of the code:
ini_set("arg_separator.output", "&");
ini_set("url_rewriter.tags", "a=href,area=href,frame=src,input=src");

And put this within the contained form:
<input type="hidden" name="PHPSESSID" value="<?php echo session_id(); ?>" />

As for the opacity and the active state, I'll have that done very soon, but those are just small things now that the contact page is finally done.

Keep the suggetsions coming, I love em!





1 user(s) are reading this topic

0 members, 1 guests, 0 anonymous users