This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Code cleaned !

Discussion in 'Techie Discussion' started by L_Plates, Jan 3, 2004.

  1. L_Plates

    L_Plates Fledgling Freddie

    Im sure some of you have seen my site before and remember the code was a mess :(

    Well.... I have cleaned it up a wee bit and applied alot more CSS to make it easier to keep running.

    The site use to have around 80 images to make the layout !! it now has 1 :) So you can get the idea of how powerful CSS can be.

    http://www.webinventions.co.uk

    All comments welcomed.
     
  2. wyrd_fish

    wyrd_fish Fledgling Freddie

    well done, although your poll is broke

    looks better than the old one...
     
  3. fatbusinessman

    fatbusinessman Fledgling Freddie

    Ok, first off (so that I don't get berated too much for this post) that's a very nice site, and I like it.

    However, I can think of a number of ways in which you could (in my opinion) improve it, certainly from the point of view of generating simpler, more readable and more maintainable code.
    • While it is of course entirely your decision, and while the method that you've got works fine, I would advise against the way you've implemented the layout. From a quick skim through your code, I've noticed that you've not just got tables within tables, but tables within tables within tables within tables within tables within tables (that's six levels of table-nesting). While cutting out table layouts altogether would probably be an excessive amount of work for the benefit it would get you, I would suggest reducing the number of nested tables you've got.
    • One way to achieve this involves the side-menus - these could be replaced fairly simply with the heading (e.g. "Portfolio site") in an <h1> tag, specified to look as you wish in the stylesheet (text colour, background color, border style and width can all be defined for block elements such as headings). The list of links could then be put within a <ul> list, which could again have its looks defined in the stylesheet. Sadly there doesn't appear to be any way to tell the list to use the ">" character as a list marker, so you might have to put this in as an image (although someone may be able to correct me on this).

    The current 'fashion' in website design is this: separate form from content. Put simply, this means that you can define nigh-on everything that actually defines how a website is displayed within your stylesheet, and have your HTML code display only the structure of the site. For instance, instead of your current code, your HTML could theoretically look something like this:
    Code:
    <html>
    <head>--head stuff--</head>
     <body>
      <div id="header"></div>
      <div id="leftside">
       <div>
        <h1>Portfolio site</h1>
        <ul>
         <li>News</li>
         --Other links--
        </ul>
       </div>
      --Other sidebars--
      </div>
      <div id="content">
      --Whatever content there is--
      </div>
      <div id="rightside">
       --Right-hand sidebars--
      </div>
      <div id="footer">
       --Footer stuff--
      </div>
     </body>
    </html>
    
    With virtually no information about how to display the content built into the HTML. This makes your code much more easily readable, theoretically faster for browsers to display (older browsers certainly don't like many levels of nested tables, I'm not sure about newer ones) and makes it much easier to maintain. You could, for instance, have both sidebars at the left and the footer at the top with just a few changes to the stylesheet.

    Sorry, ramble over. Hope this proves vaguely useful. If you'd like more details, feel free to get in touch with me (or with Jonty - he's more experienced with this than I am).
     
  4. L_Plates

    L_Plates Fledgling Freddie


    Broke its ok for me !

    Whats it doin for you ?


    -----

    Thanks for the info and ideas aswell FB. I will agree that the main flaw left in the code is the Nested tables. I have had to do that to get the desired layout using CSS in stead of images. Over time im sure i will change the layout but for the time being the tables will have to stay.

    As long as i can get rid of most junk ill be happy.

    Thnaks for the comments so far guys.
     
  5. wyrd_fish

    wyrd_fish Fledgling Freddie

    it gets an action cancled on the results page
     
  6. L_Plates

    L_Plates Fledgling Freddie

    hmm everything is ok this end *Scratches Head*

    What browser you using ?
     
  7. -=[U.D]=- Raverbaby

    -=[U.D]=- Raverbaby Fledgling Freddie

    nope, it works fine on my comp
     
  8. L_Plates

    L_Plates Fledgling Freddie

    Cool !


    Is it still the same WYRD ??
     
  9. wyrd_fish

    wyrd_fish Fledgling Freddie

    no, its all better.

    my connection was all shitty last night and so thats prolly what caused it...
     

Share This Page