Log in

View Full Version : Some constructive criticism, please?



alexjewell
07-30-2007, 11:58 PM
http://www.alexjewell.com

(password for the prompt is "al3x" - it's there until I'm fully done working on the site...I know it isn't very secure, but it's working for now)

Let me know what you think about the design, the coding, anything. I want this to be a great site, and I want to know what you think. Thanks.

tech_support
07-31-2007, 05:42 AM
Well... making the text white would be nice. I'm growing blind... :p

Also, validate (http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.alexjewell.com%2Findex.php%3Fyerface%3Dabout_design) your page.

shachi
07-31-2007, 06:29 AM
And what's up with the password? Anyone can find it out. :p

Try this in firefox:



view-source:http://www.alexjewell.com/js/pshh.js

Trinithis
07-31-2007, 08:19 PM
I don't like the use of the iframe in your design section. Just make it a part of the page, where the page itself scrolls instead of the frame.

Also, if you don't know the password, you are stuck in an endless loop of prompts. I had to task manager to exit my browser.

Sliight
07-31-2007, 08:45 PM
I think the password is just there to lock out general people until it's public ready...ya?

alexjewell
08-01-2007, 02:04 AM
The password really isn't meant to be secure, it's just a temporary block until I'm ready to "release" the site. It keeps looping until the right password is entered because that's the whole point, the content is supposed to be hidden until the password is entered. Any ideas on how to allow people to close the window if they don't know the password? Maybe a key they can hit to disable the prompt and refresh to an error page? Also, it will only be up for a little while longer - it was just a quick idea.



Well... making the text white would be nice. I'm growing blind...

Also, validate your page.


Haha, Tech. Maybe I'll add a little something just for you. We'll see.

And the validation - the only thing that doesn't validate is a target="_blank" in a link...I'll probably replace it with a javascript function.


I don't like the use of the iframe in your design section. Just make it a part of the page, where the page itself scrolls instead of the frame.

The iframe is temporary. I plan on condensing my design site into this one, and when I do that I'll just make a portfolio within this site. Until then, I have the iframe. Trust me - it isn't my favorite part of the site, either, but oh well.

Thanks guys for the comments/ideas.

Twey
08-01-2007, 05:13 AM
Any ideas on how to allow people to close the window if they don't know the password? Maybe a key they can hit to disable the prompt and refresh to an error page?That's what the cancel button is for. The prompt() function returns null on cancel:
var r;
while((r = prompt("Password:")) !== "al3x")
if(r === null)
location.href = "http://www.google.com/";

webby
08-05-2007, 03:13 PM
loose the black text on that dark putty green. It's too hard to read.

Twey
08-05-2007, 04:06 PM
Loose \Loose\ (l[=oo]s), v. n. [imp. & p. p. Loosed (l[=oo]st); p. pr. & vb. n. Loosing.] [From Loose, a.]

1. To untie or unbind; to free from any fastening; to remove the shackles or fastenings of; to set free; to relieve.I suspect you probably meant "lose."

webby
08-31-2007, 05:49 PM
yes... LOL

Trinithis
09-01-2007, 12:37 AM
Your http://www.alexjewell.com/imgs/loginbutt.gif looks funny with the cut off cursive. Either expand the box it's in or make the letters float out of the box.

Also, when you switch through the resume/pricing/portfolio tabs, make it so that the page does not jump back to the top; keep the window scroll bar at the same height.

thesimplecreator
09-01-2007, 10:45 PM
this might be off topic but your prices seem alittle crazy
but hey it might be just me im cheap

Twey
09-01-2007, 11:48 PM
No, it's about right -- I charge around £20 (~$40) per hour, which works out about the same in most cases.

boxxertrumps
09-02-2007, 04:29 AM
I make around $6CAN/h washing dishes...

Now i remember why i got into web design.