2 {{ upvoteCount | shortNum }}
2 {{ upvoteCount | shortNum }}

View Source Code

by RickyLM
1 replies
  • WEB DESIGN
    • |
Hello members,

I would appreciate if one of you savy web designers would look at my website and see what is missing in my web design. I would like to know if it is search engine friendly or if there is any improvements I can make to the code. I can take criticism so fire away.

Power Up Your Marketing
#code #source #view
  • Profile picture of the author Tomos Wyn
    Hey,

    I've just taken a look at your source code, and here's a couple of things I came accross:

    1: I'd move your styles into a seperate .css file, makes the source neater and makes editing a lot easier.

    2: Remove inline styling. For example, you could change this:

    HTML Code:
    <p style="margin-left: 40px; margin-right: 40px" align="center"><span style="font-family: 'Arial','sans-serif'; color: #333333; font-size: 12pt" hasbox="2"><strong><font color="#000000">12 SC&nbsp;has everything you need to run your email campaigns</font><font color="#000000">. </font></strong></span></p>
    to:

    HTML Code:
    <p class="center"><span class="somestyle" hasbox="2">12 SC&nbsp;has everything you need to run your email campaigns</span></p>
    and in your css add:

    Code:
    .center {
text-align: center;
margin-left: 40px;
margin-right: 40px;
}

.somestyle {
font-family: 'Arial','sans-serif'; 
color: #333333;
font-size: 12pt;
}
    3: Don't use tables to create your layouts. It is my opinion that tables have a place, but for tabular data, such as information tables. Use div's for building your layout.

    4: Improve your search engine friendliness by using heading tags (<h1>, <h2> e.t.c) for important pieces of content. For example, on line #28, you could change:

    HTML Code:
    <table style="background-color: #011b40; width: 90%; height: 50px" border="1" cellspacing="0" cellpadding="0"> 
<tbody> 
<tr> 
<th> 
<center><font color="#ffffff" size="6">&nbsp;</font><font color="#ffffff" size="5"><span style="font-size: 24pt"> If you do not have an <font color="#ff0000">Autoresponder</font> your competition will leave you behind!</span></font></center></th></tr></tbody></table>
    to:

    HTML Code:
    <h1>If you do not have an <span>Autoresponder</span> your competition will leave you behind!</h1>
    and in your css add:

    Code:
    h1 {
font-size: 24px;
text-align: center;
color: #ffffff;
}

h1 span {
color: #ff0000;
}
    5: In general, tidy things up and get some consistency going on. For example here:

    http://localhostr.com/files/e4e647/S...2019.54.51.png

    Your spacing is different for the two elements, which sits uncomfortable with my eyes. Either tie it in to make it flush with the paragraph above, or if your trying to catch the user's eye, maybe move the content into the center a bit more, and add some background color to it to differentiate and catch the eye.


    Hope this helps a bit
    {{ DiscussionBoard.errors[2409513].message }}

Trending Topics