PhotoClickr Posted September 15, 2007 Share Posted September 15, 2007 I have almost finished my site and would like you all to go and check it out. Let me know of any bugs or security holes please... My site is: http://www.photoclickr.com Thank you all... It is much appreciated as this is my first real site.... Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/ Share on other sites More sharing options...
TheHeretic Posted September 15, 2007 Share Posted September 15, 2007 Like your button roll overs, easy site to navigate. Maybe a tad too many different fonts on the home page and you need an eye catching banner imho. Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-348933 Share on other sites More sharing options...
PhotoClickr Posted September 15, 2007 Author Share Posted September 15, 2007 Thanks for the review... I am working on banners as we speak. Just a question though... When you say a tad too many fonts.... Does that mean, too many different sizes because as far as I know, the font should be the same for everything on the site. I have the font set so everything is the same. Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-348997 Share on other sites More sharing options...
moberemk Posted September 15, 2007 Share Posted September 15, 2007 Not too bad; but for the big picture banner in the middle (which honestly strikes me as superfluous), you know the blue borders on the sides? If you're going to have them, try making them continue throughout the design, through the header picture and the footer even-that might help make the site a bit more focused. Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-349020 Share on other sites More sharing options...
Daniel0 Posted September 16, 2007 Share Posted September 16, 2007 http://www.photoclickr.com/about.html Why do you "center" the text (before/after) using a lot of non breaking spaces? Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-349450 Share on other sites More sharing options...
PhotoClickr Posted September 16, 2007 Author Share Posted September 16, 2007 Ok I have fixed that. No more spaces Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-349580 Share on other sites More sharing options...
AXiSS Posted September 16, 2007 Share Posted September 16, 2007 Nice attempt at Web 2.0 design, it seems you did some pretty good research. Things I'd recommend are finding something better to put there than the big image bar, with the green and stuff, and remove the blue sides. Also, you should get a matching Web2.0 logo, and I doubt the legality of using the Disney/Faux-Disney font. Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-349604 Share on other sites More sharing options...
dbrimlow Posted September 17, 2007 Share Posted September 17, 2007 Nice attempt. But PLEASE validate your code. You have 29 errors (most due to not closing tags - AS REQUIRED BY XHTML). People should always check their code at the w3c validator - here's yours: http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.photoclickr.com%2F Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-350095 Share on other sites More sharing options...
TheFilmGod Posted September 17, 2007 Share Posted September 17, 2007 Looks actually pretty good. I agree with all previous comments. So no point repeating, the only comment worth repeating is the one asking the legality of using the disney font...? Is that legal? Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-350135 Share on other sites More sharing options...
dbrimlow Posted September 20, 2007 Share Posted September 20, 2007 PhotoClickr, you now increased the errors to 38 because you are not closing your tags correctly. XHTML REQUIRES all tags to have an open and close. Stand alone tags like metas, images and break tags close with a forward slash. all meta tags must be closed, like this: you have: <meta name="ROBOTS" content="ALL"> should be: <meta name="ROBOTS" content="ALL" /> Your first few tags are right, but then it goes bad. Same with graphics. <a href="http://www.photoclickr.com" alt="online photo services"> should be: <a href="http://www.photoclickr.com" alt="online photo services" /> Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-351628 Share on other sites More sharing options...
jcombs_31 Posted September 20, 2007 Share Posted September 20, 2007 It looks like you tore up a template. The header / logo needs some work. needs to be sharp. The blue side bars seem to end short. The menu is in a strange place. The colors are nice and bright but something needs to hold it all together. Quote Link to comment https://forums.phpfreaks.com/topic/69431-could-you-all-please-review-me/#findComment-351721 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.