[kod css/xhtml] kod xHTML Opcje

[sebekzosw]

Proszę o ocenę kodu xHTML

Link

[webber]

Pierwszy spojrzenie i... divitis. Ale jedziemy po kolei.

Kod

class="floatleft"

Ciekawa nazwa klasy (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Kod

<a href="index.html"><img src="images/aktualnosci.jpg" alt="Aktualności" class="floatleft" /></a>
            <a href="filmy.html"><img src="images/filmy.jpg" alt="Aktualności" class="floatleft" /></a>
            <a href="wycinki.html"><img src="images/wycinki_prasowe.jpg" alt="Aktualności" class="floatleft" /></a>

            <a href="linki.html"><img src="images/linki.jpg" alt="Aktualności" class="floatleft" /></a>

To jest lista.

A zamiast stosować klasę floatleft dałbyś div#logoikony img { float: left; }

Kod

<div id="logo">
        <div id="tytul_logo">Zapisy</div>
        <div id="tresc_logo">Przykładowy tekst cd; Przykładowy tekst cd; Przykładowy tekst cd; Przykładowy tekst cd :-)</div>
    </div>

Myślę że lepiej było się posłużyć znacznikami h i p i zamiast dawać kolejne divy i id dać div#logo p { styl }, div#logo h2 { styl }

Kod

<div id="menu">
        <ol class="menulist">
            <li><a href="index.html">Aktualności</a></li>
            <li><a href="http://forum.tkd-bielsko.pl/">Forum</a></li>
            <li><a href="filmy.html">Filmy</a></li>

            <li><a href="galeria.html">Galeria</a></li>
            <li><a href="Zapisy.xhtml">Treningi</a></li>
        </ol>

        <div class="text1">
            przykładowa tresc
        </div>
        <div class="text2">
            przykładowa tresc
            <div class="linia"></div>

        </div>
    </div>

Do menu mogłeś odwołać się poprzez div#menu ol { }, a te divy o klasie text1 i 2 eh... masakra (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Kod

<div class="newsbar">
            <div class="newstitle">
                Lorem ipsum dolor sit amet
            </div>

            <div class="newsinfo">
                <div class="zegar"></div>
                28/04/2009, 13:46
            </div>
        </div>
        <div class="newsinside">
            Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed vel velit sed odio aliquet suscipit. Integer tincidunt augue at dui. Morbi leo. Quisque quis sem sed ipsum rutrum aliquam. Suspendisse potenti. Suspendisse potenti. Praesent dapibus adipiscing justo. Sed consequat. Quisque consequat. Morbi venenatis dapibus libero. Nam nec nisl. Duis quis purus non nibh varius viverra. Class aptent taciti sociosqu ad litora torquent per conubia nostra, per inceptos himenaeos. In facilisis, urna et imperdiet tempus, mauris tellus ornare tellus, id semper odio leo sed ipsum. Nunc suscipit dignissim erat. Vestibulum rhoncus magna eu dolor sagittis imperdiet. Proin aliquam. Vivamus ante arcu, faucibus non, mollis ac, scelerisque eu, metus. Nam ornare pellentesque nisi.
            <div class="clearboth"></div>
        </div>
        <div class="newsend"></div>

Znów pominąłeś h, nie używasz span a przydałoby się do zegara, no i tekst bardziej w p...

Kod

<div id="zwiazki_kluby">
            <a href="http://www.pztkd.lublin.pl/" id="obr03" class="rollover"><img src="images/png/pztkd_szary.png" alt="Polski Związek Taekwon-Do" /></a>
            <a href="http://www.ksdragon.pl/" id="obr01" class="rollover"><img src="images/png/dragon_szary.png" alt="Klub Sportowy Dragon" /></a>
            <a href="http://www.tkd-itf.org/" id="obr02" class="rollover"><img src="images/png/itf_szary.png" alt="International Taekwon-Do Federation" /></a>
        </div>

To też jest lista. Dodaj width i height do obrazków.