Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 12 of 12
  1. #1
    Regular Coder
    Join Date
    Mar 2011
    Posts
    212
    Thanks
    10
    Thanked 1 Time in 1 Post

    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.

  • #2
    Regular Coder
    Join Date
    Mar 2011
    Posts
    212
    Thanks
    10
    Thanked 1 Time in 1 Post
    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.

  • #3
    New Coder
    Join Date
    Apr 2012
    Location
    Mumbai , India.
    Posts
    12
    Thanks
    1
    Thanked 3 Times in 3 Posts
    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..

  • #4
    Regular Coder
    Join Date
    Mar 2011
    Posts
    212
    Thanks
    10
    Thanked 1 Time in 1 Post
    ugh, my apologies, I was truly not awake this morning haha. www.trendstudiosorlando.com/test2/index.html

  • #5
    Regular Coder
    Join Date
    Mar 2011
    Posts
    212
    Thanks
    10
    Thanked 1 Time in 1 Post
    pretty please?

  • #6
    Senior Coder
    Join Date
    Apr 2010
    Posts
    1,420
    Thanks
    68
    Thanked 102 Times in 101 Posts
    #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 7 years. My business:
    American Made Signs

  • #7
    Regular Coder
    Join Date
    Mar 2011
    Posts
    212
    Thanks
    10
    Thanked 1 Time in 1 Post
    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.

  • #8
    The fat guy next door VIPStephan's Avatar
    Join Date
    Jan 2006
    Location
    Halle (Saale), Germany
    Posts
    8,636
    Thanks
    6
    Thanked 1,003 Times in 976 Posts
    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.
    Last edited by VIPStephan; 04-17-2012 at 06:14 PM.

  • #9
    Regular Coder
    Join Date
    Mar 2012
    Posts
    168
    Thanks
    5
    Thanked 11 Times in 11 Posts
    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.

  • #10
    Regular Coder
    Join Date
    Mar 2011
    Posts
    212
    Thanks
    10
    Thanked 1 Time in 1 Post
    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!

  • #11
    Senior Coder
    Join Date
    Apr 2010
    Posts
    1,420
    Thanks
    68
    Thanked 102 Times in 101 Posts
    Been a sign maker for 7 years. My business:
    American Made Signs

  • #12
    New to the CF scene
    Join Date
    May 2012
    Posts
    2
    Thanks
    0
    Thanked 0 Times in 0 Posts
    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


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •