...

View Full Version : Need a site review before I put this live



lmorales
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.

Thanks!

www.trendstudiosorlando.com/test2/index.html

lmorales
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.

bzforum
04-16-2012, 06:19 PM
Where is the URL to the site?? Did I miss something?

lmorales
04-16-2012, 07:07 PM
ugh, my apologies, I was truly not awake this morning haha. www.trendstudiosorlando.com/test2/index.html

lmorales
04-17-2012, 04:08 PM
pretty please? :)

myfayt
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.

lmorales
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.

Thank you!

VIPStephan
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.


That fixed navigation is gonna be a problem if the viewport isn’t large enough. It has the perfect height for my 1280800 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 (http://css-tricks.com/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).
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>
<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.

KULP
04-17-2012, 08:17 PM
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.

lmorales
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!

myfayt
04-18-2012, 04:14 AM
http://www.w3schools.com/tags/tag_doctype.asp

buyfblike
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



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum