Coding issue - help!

Discussion in 'HTML & Website Design' started by belenenses, Nov 26, 2008.

  1. #1
    Hi Guys,

    Can someone help resolve this small but significant coding issue, please? Please visit
    http://www.walking-languedoc.com/

    firstly using Explorer and then in Firefox. It seems to be a border problem, and must be a simple coding issue, but I cannot see it. Any ideas?

    Many thanks,

    B.
     
    belenenses, Nov 26, 2008 IP
  2. kk5st

    kk5st Prominent Member

    Messages:
    3,497
    Likes Received:
    376
    Best Answers:
    29
    Trophy Points:
    335
    #2
    kk5st, Nov 26, 2008 IP
  3. i.run.shit

    i.run.shit Peon

    Messages:
    61
    Likes Received:
    4
    Best Answers:
    0
    Trophy Points:
    0
    #3
    In your header you have
    <link rel="stylesheet" type="text/css" href="http://www.walking-languedoc.com.css">
    Code (markup):
    which could be presenting a problem because it's not linking to anywhere that exists.

    There are quite a few flaws in your CSS. The CSS that's directly inserted into your "head" tag
    <style type="text/css" media="screen">
    #topimage { 
     background-image: url('http://www.walking-languedoc.com/pics/Ceze/River_1.jpg'); 
    	 }
    
    body table#MBL_COMM {border:px solid #FF5200;}
    body table#MBL_COMM td.mbl_img {background-color:#FFFADD;}
    body table#MBL_COMM img {border-color:blue;}	 
    	 
    </style>
    Code (markup):
    You don't have any number for your border for the #MBL_COMM section, that's not good.

    There are a a lot of errors in your walking-languedoc.css file. First off #WHITE is not a proper value for a color.

    Your
    </noscript>
    Code (markup):
    is a closing tag for somethign that's not open. Your meta and br elements should be closed ending with a /> not just a >

    I think I found the problem you're needing a solution for though. Line 120 -
    <a href="http://www.walking-languedoc.com/Hiking_Provence_Hills_and_Valleys.html"><img src="pics/Ceze/Aujaguet panorama.jpg" border="0" width="570" height="188" alt="Walking Provence" align="middle"></div> </a>
    Code (markup):
    remove the div tag and that should do it honestly. I can't be positive because your code is horrible. All in all you have 108 HTML warnings on your site. Most of them are because you don't close your tags or you close them too many time.
     
    i.run.shit, Nov 26, 2008 IP
  4. belenenses

    belenenses Peon

    Messages:
    24
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #4
    Hi Guys,

    I've tried both of your suggestions without success. Any other ideas? I havent a clue about css and 'borrowed' the one I'm using and had it amended by someone to suit.

    However, I have not changed it since it was given me a year ago and so I dont think the problem is there. I changed other things on ythe Homepage yesterday and only checked them in Explorere before uploading. So I think ther coding problem is there.

    Best wishes,

    B.
     
    belenenses, Nov 26, 2008 IP
  5. i.run.shit

    i.run.shit Peon

    Messages:
    61
    Likes Received:
    4
    Best Answers:
    0
    Trophy Points:
    0
    #5
    Diddle around with the float properties on your .sidemenu class. You've already got your float property on your your #left-column and there's no need for another float property within. Also, float your right column to the left as well. Let me know how it turns out. ;)
     
    i.run.shit, Nov 26, 2008 IP
  6. kk5st

    kk5st Prominent Member

    Messages:
    3,497
    Likes Received:
    376
    Best Answers:
    29
    Trophy Points:
    335
    #6
    No, you didn't. The fix I gave you works.

    Open your style sheet in your editor, and find the selector, "#outer". Add "overflow: hidden;" to the ruleset there.

    I also linked to an explanation of just how IE's buggy behavior encourages buggy coding.

    gary
     
    kk5st, Nov 26, 2008 IP
  7. belenenses

    belenenses Peon

    Messages:
    24
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #7
    Hi Gary,

    I di what you said yesterday and have repeated it again now. The coding now reads:
    #outer {width: 770px; overflow: hidden;
    margin-left:auto; margin-right:auto;
    background-color:#FFFFFF; color: inherit;

    ...but the problem is not resolved. Am I missing something?

    Thanks,

    B.
     
    belenenses, Nov 27, 2008 IP
  8. kk5st

    kk5st Prominent Member

    Messages:
    3,497
    Likes Received:
    376
    Best Answers:
    29
    Trophy Points:
    335
    #8
    This is what you have,
    #outer {  /*walking-...uedoc.css (line 29)*/
    background-color:#FFFFFF;
    color:inherit;
    margin-left:auto;
    margin-right:auto;
    width:770px;
    }
    Code (markup):
    If you're adding to another copy of the page, I need to see it in order to help you. Adding the overflow prperty to the above does, absolutely, fix your issue.

    gary
     
    kk5st, Nov 27, 2008 IP
  9. belenenses

    belenenses Peon

    Messages:
    24
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #9
    Hi Gary,

    This is very strange, as we seem to be looking at two separate pages. The css was uploaded ages ago and I forget where it should be placed. I am assuming its url is:
    http://walking-languedoc.com/css/enlightened-traveller.css
    and that it is uploaded (using Filezilla) to public_html/css

    although there is a second copy also uploaded at in the public_html directory itself. Either way, they both read …

    #outer {width: 770px; overflow: hidden;
    margin-left:auto; margin-right:auto;
    background-color:#FFFFFF; color: inherit;

    And neither reads…

    #outer { /*walking-...uedoc.css (line 29)*/
    background-color:#FFFFFF;
    color:inherit;
    margin-left:auto;
    margin-right:auto;
    width:770px;
    }

    I’d appreciate your input to resolve this matter issue.

    Best wishes,

    B.
     
    belenenses, Nov 27, 2008 IP
  10. kk5st

    kk5st Prominent Member

    Messages:
    3,497
    Likes Received:
    376
    Best Answers:
    29
    Trophy Points:
    335
    #10
    Ah, but that's not the stylesheet your page links.
    <link rel="stylesheet" type="text/css" 
    href="http://www.walking-languedoc.com.css">
    
    <link rel="stylesheet" type="text/css" href="w3buttons.css" media="screen" >
    <link rel="stylesheet" type="text/css" href="walking-languedoc.css">
    Code (markup):
    cheers,

    gary
     
    kk5st, Nov 28, 2008 IP
  11. belenenses

    belenenses Peon

    Messages:
    24
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #11
    belenenses, Nov 28, 2008 IP
  12. belenenses

    belenenses Peon

    Messages:
    24
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #12
    Dear Gary,

    Talk about bad hair day, I must have been having a whole week of it! Can you believe I was making changes to the .css of my second web site?! I need to lie down a bit.

    Anyway, I've now made the changes you suggested to the correct stylesheet and the site looks beautiful once more thanks to you and your patience!

    I'm a case in question for your mantra about people building a web site without knowing what they're doing - we learn by doing, true, but we need to get a bit more sleep between lessons!

    Cheers!

    B.
     
    belenenses, Nov 28, 2008 IP