Jump to content

Archived

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

sdaniels

my personal website

Recommended Posts

Im a novice with php/mysql. I have built myself a site for testing/practice purposes. Id just like general comments on the look/feel of the site before I try to build sites for other people. Thanks.

http://www.shaundaniels.com

Share this post


Link to post
Share on other sites
There are a few things about it ->

I commend your use of graphics trying to make it look different - but the dark colours dont really work for me.
The HOME page is completely empty.
Your top logo/graphic is poor and needs work.

Your Contact input box is WAY down the page -> move it up so I dont have to be scrolling at 1024 x 768

Your picture gallery is a tad bare. I also dont like it opening a new window to show a large image.

Your login doesnt show much. Like even anybody not even logged in can see -> http://www.shaundaniels.com/webcam.php

Anyways - a chnage of colour and top div are needed IMO.

-steve

Share this post


Link to post
Share on other sites
good call on the text box. my resolution in 1280 x 1024 and I always forget to check for differences.
good call on the webcam page too, Ill have to look at my conf file. if I cant find anything there then ill add some session validation to my pages.

These are the responces i was looking for. Thanks.

logo is a placeholder for now... content is sparce on main page for now, im working on functionality first.
colors stay, Its easy on my eyes. If i buid pages for others then I will keep the color issue in mind.

what is this IMO I keep hearing about?

thanks for the review.

Share this post


Link to post
Share on other sites
do you know if you can set the conf file in appache to only allow access to index.php in the htdocs folder? or am i going to have to validate session variables on each page?

Share this post


Link to post
Share on other sites
nevermind the conf question, im just validating session variables now. The webcam.php issue is fixed.

Share this post


Link to post
Share on other sites

×

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.