Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 3 of 3
  1. #1
    New to the CF scene
    Join Date
    Dec 2013
    Posts
    2
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Take a look at my codes

    Hi,

    I mimicked this website for a project and I just wanted to know how my codes are. I am a noob in HTML coding so my codes may be sloppy and really bad. Suggestion and feedback is welcome to fix or make it more neat (+indentation). Thanks!


    Code:
    <!DOCTYPE html>
    <html>
       <head>
          <title>Game of Thrones</title>
          <meta name="description" content="Game of Thrones">
          <meta charset="utf-8">
          <link href="style/normalize.css" rel="stylesheet" type="text/css">
          <link href="style/main.css" rel="stylesheet" type="text/css">
          <!--[if lt IE 9]>
          <script type="text/javascript" src="script/modernizr.custom.19100.js"> 
          </script><![endif]-->
       </head>
       <body>
          <div id="all-wrapper">
          <header>
          <nav id="nav">
             <ul>
                <li><a id="active" href="index.html" title="Home">Home</a></li>
                <li><a href="dawnage.html" title="The Dawn Age">Dawn Age</a></li>
                <li><a href="ageofheroes.html" title="The Age of Heroes">Age of Heroes</a></li>
                <li><a href="lannisters.html" title="History of the Lannisters">Lannisters</a></li>
                <li><a href="starks.html" title="History of the Starks">Starks</a></li>
                <li><a href="targaryens.html" title="History of the Targaryens">Targaryens</a></li>
                <li><a href="tullys.html" title="History of the Tullys">Tullys</a></li>
             </ul>
          </nav>
          <div id="headerbg">
             <h1><a href="index.html" id="logo"></a></h1>
          </div>
          <div id="borderline"></div>
          </header>
    
          <div id="container">
          <div id="sidenav">
          <nav>
             <ul class="leftul">
                <li class="sub-nav-top"><a class="active" href="index.html" title="Home">Home</a></li>
                <li><a href="http://www.hbo.com/game-of-thrones" title="">Official Page</a></li>
                <li><a href="http://www.winteriscoming.net" title="">Fanpage</a></li>
                <li><a href="http://www.amazon.com/Game-Thrones-Song-Fire-Book/dp/0553593714/" title="">Buy The Book</a></li>
                <li><a href="http://stuffpoint.com/game-of-thrones/images/wallpapers/" title="">Wallpapers</a></li>
                <li class="sub-nav-bottom"><a href="http://westeros.org/" title="">Learn More</a></li>
             </ul>
          </nav>
    
          <div id="quote">
             <p>“Blah Blah">— Hank Stuever</p>
          </div>
    
             <p id="media"><a href="http://www.youtube.com/watch?v=8ixEWrTLiZg" target="_blank"><img alt="Youtube" src="images/youtube.jpg" title="Youtube"></a> <a href="https://www.facebook.com/GameOfThrones" target="_blank"><img 
    
    alt="Facebook" src="images/facebook.jpg"
    title="Facebook"></a> <a href="https://twitter.com/GameOfThrones" target="_blank"><img alt="Twitter" src="images/twitter.jpg" title="Twitter"></a></p>
          </div>
    
          <div id="container2">
             <p id="header2"><img alt="" src="images/h2.jpg" title=""></p>
    
          <section>
             <h2>Header 2</h2>
             <p class="format">Blah Blah</p>
    
             <h2>Overview</h2>
             <p class="format">Blah Blah</p>
    
             <h2>Plot</h2>
             <p class="format">Blah Blah</p>
    
             <h2>Cast and characters</h2>
             <p class="format">Blah Blah.</p>
    
             <p id="source">Reference: <a href="http://en.wikipedia.org/wiki/Game_of_Thrones" target="_blank">Wikipedia</a></p>
          </section>
          </div>
          </div>
    
          <footer class="footer">
          <p>&copy; Game of Thrones, All Rights Reserved. Website Design by <a href="mailto:mailme@gmail.com">Blah Blah</a></p>
          </footer>
          </div>
       </body>
    </html>

  • #2
    Senior Coder
    Join Date
    Aug 2006
    Posts
    1,208
    Thanks
    7
    Thanked 268 Times in 267 Posts
    Looks very nice. I would only note a couple css things.
    Code:
          <nav id="nav">
          <footer class="footer">
    I will guess that you only have one <nav> and one <footer> section planned for this page. Just style <nav> and <footer>, and not create new ID's and classes for these two.

  • #3
    New to the CF scene
    Join Date
    Dec 2013
    Posts
    2
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by tracknut View Post
    Looks very nice. I would only note a couple css things.
    Code:
          <nav id="nav">
          <footer class="footer">
    I will guess that you only have one <nav> and one <footer> section planned for this page. Just style <nav> and <footer>, and not create new ID's and classes for these two.

    Thanks, I reworked some of the codes and html to make it more neater.


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •