Brand New Landing Page Debuted

wwag · 06-23-2011, 01:10 AM

Hey everyone, we just spent 2 days re-designing the landing page for SideSkills.com. What do you think?

ronaldroe · 06-23-2011, 02:36 AM

The layout is nice, clean and professional. The code, on the other hand, needs help. Don't use tables for layout, even for forms. It's not what they were meant for, bloats your code, makes it harder for search engines to index the page, and makes structural changes a royal pain. Forms are actually pretty easy to style using standard, semantic code.

Move your CSS and JScript to external files. Again, more bloated code.

Your page has a body nested within a body. Not sure why that is, but it couldn't have been on purpose.

Move
<link rel="stylesheet" type="text/css" href="/css/wowwindow.css" />
within the head tags.
Also, if you MUST have CSS within style tags, place that in the head as well.

wwag · 06-23-2011, 04:31 AM

That everything you said about the code is correct, we rushed it to get it live and tomorrow we're moving things into files...you are very thorough...thanks!

reverseengineer · 06-28-2011, 10:59 PM

I have nothing to critique about the design, I just wanted to say that the crowd of people above the footer is very creative, I like that. You had to move it quick, have you designed other sites before?

06-23-2011, 01:10 AM	#1
wwag New Member Join Date: Feb 2010 Location: Los Angeles Posts: 17	Brand New Landing Page Debuted - SideSkills.com Hey everyone, we just spent 2 days re-designing the landing page for SideSkills.com. What do you think? __________________ WebWorks & Graphics, SideSkills.com Creative and Functional Web Development http://www.webworksandgraphics.com http://www.sideskills.com

06-23-2011, 02:36 AM	#2
ronaldroe Platinum Member Join Date: Jan 2011 Location: Oklahoma City, OK Posts: 677	The layout is nice, clean and professional. The code, on the other hand, needs help. Don't use tables for layout, even for forms. It's not what they were meant for, bloats your code, makes it harder for search engines to index the page, and makes structural changes a royal pain. Forms are actually pretty easy to style using standard, semantic code. Move your CSS and JScript to external files. Again, more bloated code. Your page has a body nested within a body. Not sure why that is, but it couldn't have been on purpose. Move <link rel="stylesheet" type="text/css" href="/css/wowwindow.css" /> within the head tags. Also, if you MUST have CSS within style tags, place that in the head as well. __________________ Ronald Roe Small Business Web Design

06-23-2011, 04:31 AM	#3
wwag New Member Join Date: Feb 2010 Location: Los Angeles Posts: 17	Should have said... That everything you said about the code is correct, we rushed it to get it live and tomorrow we're moving things into files...you are very thorough...thanks! __________________ WebWorks & Graphics, SideSkills.com Creative and Functional Web Development http://www.webworksandgraphics.com http://www.sideskills.com

06-28-2011, 10:59 PM	#4
reverseengineer Silver Member Join Date: Jan 2011 Location: San Diego, CA Posts: 136	I have nothing to critique about the design, I just wanted to say that the crowd of people above the footer is very creative, I like that. You had to move it quick, have you designed other sites before? __________________ Reverse Engineering Faro, Romer, & MicroScribe

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode