View Full Version : Need a site review before I put this live
04-16-2012, 05:31 PM
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.
04-16-2012, 06:09 PM
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.
04-16-2012, 06:19 PM
Where is the URL to the site?? Did I miss something?
04-16-2012, 07:07 PM
ugh, my apologies, I was truly not awake this morning haha. www.trendstudiosorlando.com/test2/index.html
04-17-2012, 04:08 PM
pretty please? :)
04-17-2012, 04:22 PM
#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.
04-17-2012, 06:18 PM
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.
04-17-2012, 07:11 PM
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.
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.
You have a few validation errors on all or most of your pages.
Write semantic HTML (http://en.wikipedia.org/wiki/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.
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:
<h2>Trend Mani-Couture Services</h2>
<th scope="row">Exclusive Spa Manicure</th>
<th scope="row">Luxurious Classic Manicure</th>
<th scope="row">Express Manicure</th>
<th scope="row">Sleek Polish Change</th>
<h2>Add on Services</h2>
<th scope="row">Paraffin Hot Wax Treatment</th>
<th scope="row">Flirty French Tips</th>
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.
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.
04-17-2012, 10:26 PM
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!
04-18-2012, 04:14 AM
05-03-2012, 10:31 AM
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