Some constructive criticism, please?

[alexjewell]

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]

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

Also, validate your page.

[shachi]

And what's up with the password? Anyone can find it out.

Try this in firefox:

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

[Trinithis]

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]

I think the password is just there to lock out general people until it's public ready...ya?

[alexjewell]

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]

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]

loose the black text on that dark putty green. It's too hard to read.

[Twey]

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]

yes... LOL