Go Back   CodingForums.com > :: Client side development > General web building > Site reviews

Before you post, read our: Rules & Posting Guidelines

Reply
 
Thread Tools Rate Thread
Enjoy an ad free experience by logging in. Not a member yet? Register.
Old 04-16-2012, 04:31 PM   PM User | #1
lmorales
Regular Coder

 
Join Date: Mar 2011
Posts: 192
Thanks: 8
Thanked 1 Time in 1 Post
lmorales is an unknown quantity at this point
Need a site review before I put this live

So far everything validates, I took some advice and make the menu static, and got rid of some annoying empty space. What Im concerned with is how the SEO is for this ( Im kinda new at that) and if it works cross browser wise.

Thanks!

www.trendstudiosorlando.com/test2/index.html

Last edited by lmorales; 04-16-2012 at 06:07 PM..
lmorales is offline   Reply With Quote
Old 04-16-2012, 05:09 PM   PM User | #2
lmorales
Regular Coder

 
Join Date: Mar 2011
Posts: 192
Thanks: 8
Thanked 1 Time in 1 Post
lmorales is an unknown quantity at this point
please? This is the first site I feel Ive coded without too much trouble ( and im damned proud of it) I just wanna see if I did it right or in my normal convoluted way.
lmorales is offline   Reply With Quote
Old 04-16-2012, 05:19 PM   PM User | #3
bzforum
New Coder

 
Join Date: Apr 2012
Location: Mumbai , India.
Posts: 12
Thanks: 1
Thanked 3 Times in 3 Posts
bzforum is an unknown quantity at this point
Where is the URL to the site?? Did I miss something?
__________________
Join my Forum : http://bzforum.in
Park2Peak Killer Deals on Ski's , Goggles etc..
bzforum is offline   Reply With Quote
Old 04-16-2012, 06:07 PM   PM User | #4
lmorales
Regular Coder

 
Join Date: Mar 2011
Posts: 192
Thanks: 8
Thanked 1 Time in 1 Post
lmorales is an unknown quantity at this point
ugh, my apologies, I was truly not awake this morning haha. www.trendstudiosorlando.com/test2/index.html
lmorales is offline   Reply With Quote
Old 04-17-2012, 03:08 PM   PM User | #5
lmorales
Regular Coder

 
Join Date: Mar 2011
Posts: 192
Thanks: 8
Thanked 1 Time in 1 Post
lmorales is an unknown quantity at this point
pretty please?
lmorales is offline   Reply With Quote
Old 04-17-2012, 03:22 PM   PM User | #6
myfayt
Senior Coder

 
Join Date: Apr 2010
Posts: 1,156
Thanks: 46
Thanked 95 Times in 94 Posts
myfayt can only hope to improve
#1. Very basic website, which can be good and bad. It's good for being easy to read and navigate, bad because the search engines aren't going to pick it up.

#2. Your logo pushes all the content down the page. This is going to be a nightmare for people with low screen res and mobile phones. Also the purple is horrible with the black.

#3. On the tan page you have purple text on black, this is hard to read and very bad. Use a yellow or orange.

#4. Your sexy model, if this isn't a person you tanned at your studio, remove it. If it's a random person like from google, it's illegal to display on your website. Same with the other people.

That's about all I can think of.
__________________
Been a sign maker for 5 years. My business:
American Made Signs
myfayt is offline   Reply With Quote
Old 04-17-2012, 05:18 PM   PM User | #7
lmorales
Regular Coder

 
Join Date: Mar 2011
Posts: 192
Thanks: 8
Thanked 1 Time in 1 Post
lmorales is an unknown quantity at this point
the sexy tan person is our model, all the photography on this website was taken in the studio by a photographer, as for the colors... i tend to agree, but I didnt actually design this site and dont really have any control over that, I just needed to make it work, it was done by my coworker and the owner ( ofcourse) is in love with it. As for the logo, do you suggest I make it smaller? How would I increase seo? I appreciate the critic, One thing I wanted to do was make it so that the content cant scroll above the menu, but I havent figured out how to do that yet.

Thank you!

Last edited by lmorales; 04-17-2012 at 05:21 PM..
lmorales is offline   Reply With Quote
Old 04-17-2012, 06:11 PM   PM User | #8
VIPStephan
The fat guy next door


 
VIPStephan's Avatar
 
Join Date: Jan 2006
Location: Halle (Saale), Germany
Posts: 7,613
Thanks: 5
Thanked 865 Times in 842 Posts
VIPStephan is a jewel in the roughVIPStephan is a jewel in the roughVIPStephan is a jewel in the rough
OK, since you said you didn’t design it I suppose you want a review on your coding? Overall it seems OK but there are a couple of issues.
  1. That fixed navigation is gonna be a problem if the viewport isn’t large enough. It has the perfect height for my 1280×800 px laptop display but only if I enlarge the viewport to almost the maximum height. It’s not gonna fit on smaller screens like netbooks. Modern handheld devices like tablets might be scaling the site but you can never be sure. So, you should implement something that makes it switch from fixed to absolute (or static) position depending on the viewport size. In modern browsers CSS media queries can be used, for other ones a JavaScript might be an option (default is static position, fixed if JS is supported and viewport large enough).
  2. You should always, always, always use a strict document type declaration. “Transitional” is for losers. You aren’t transitioning an old website with non-standardized HTML to standards compliant code, are you? So, no reason for a transitional doctype.
  3. You have a few validation errors on all or most of your pages.
  4. Write semantic HTML. Look at the ladies’ nails page and switch off your styles (in Firefox you can do “View > Page Style > No Style”); look at the content and tell me how much a luxurious classic pedicure costs. You see, there is no relation of any item to any price whatsoever, there are just items and random prices in separate lists. Semantically these should all be tables.
  5. Don’t use   to adjust your layout. Use floats, margin, padding, or text alignment where appropriate. For example on the ladies’ nails page that whole thing could be marked up as tables like so:
    Code:
    <h2>Trend Mani-Couture Services</h2>
    <table>
    	<tbody>
    		<tr>
    			<th scope="row">Exclusive Spa Manicure</th>
    			<td>$40</td>
    		</tr>
    		<tr>
    			<th scope="row">Luxurious Classic Manicure</th>
    			<td>$35</td>
    		</tr>
    		<tr>
    			<th scope="row">Express Manicure</th>
    			<td>$25</td>
    		</tr>
    		<tr>
    			<th scope="row">Sleek Polish Change</th>
    			<td>$15</td>
    		</tr>
    	</tbody>
    </table>
    <h2>Add on Services</h2>
    <table>
    	<tbody>
    		<tr>
    			<th scope="row">Paraffin Hot Wax Treatment</th>
    			<td>$15</td>
    		</tr>
    		<tr>
    			<th scope="row">Flirty French Tips</th>
    			<td>$10</td>
    		</tr>
    	</tbody>
    </table>
    …
    …
    You can style the table header cells (th) and the data cells (td) differently, i. e. assign a certain width to them and/or give them some padding or right-align the right column in the tables. This way the data looks just like now but has a clear relation to each other. And no redundant and meaningless non-breaking space characters are there. The same counts for all the other pages.
__________________
Don’t click this link!

Last edited by VIPStephan; 04-17-2012 at 06:14 PM..
VIPStephan is offline   Reply With Quote
Old 04-17-2012, 07:17 PM   PM User | #9
KULP
Regular Coder

 
Join Date: Mar 2012
Posts: 166
Thanks: 5
Thanked 11 Times in 11 Posts
KULP is an unknown quantity at this point
I'd probably make the background solid dark grey and then move the logo up to the top right above the navigation. Then maybe add a border around the image in the middle, maybe even slightly curved edges.
KULP is offline   Reply With Quote
Old 04-17-2012, 09:26 PM   PM User | #10
lmorales
Regular Coder

 
Join Date: Mar 2011
Posts: 192
Thanks: 8
Thanked 1 Time in 1 Post
lmorales is an unknown quantity at this point
Awesome, thanks for the critic.. Couple of questions, what the difference between strict and transitional doc types? Also, how would I style the table to look just like the page looks right now ( ill get *****ed at if its not..) Y'see, Im really more of a print designer, but since I know a lil about web, I kinda got thrown into the whole ( i need a website in a week) crap, Im trying to learn on my own, and i think im getting better, but its a longggg road haha.

Thanks for the help!
lmorales is offline   Reply With Quote
Old 04-18-2012, 03:14 AM   PM User | #11
myfayt
Senior Coder

 
Join Date: Apr 2010
Posts: 1,156
Thanks: 46
Thanked 95 Times in 94 Posts
myfayt can only hope to improve
http://www.w3schools.com/tags/tag_doctype.asp
__________________
Been a sign maker for 5 years. My business:
American Made Signs
myfayt is offline   Reply With Quote
Old 05-03-2012, 09:31 AM   PM User | #12
buyfblike
New to the CF scene

 
Join Date: May 2012
Posts: 2
Thanks: 0
Thanked 0 Times in 0 Posts
buyfblike is an unknown quantity at this point
if your code is approved by w3schools validator than it works fine with all browser and about seo just focus on content .. it must be unique
buyfblike is offline   Reply With Quote
Reply

Bookmarks

Jump To Top of Thread


Thread Tools
Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT +1. The time now is 09:09 AM.


Advertisement
Log in to turn off these ads.