[www] cięcie i kodowanie firmowy lay Opcje

[Artu123]

Witam!

Mam do oceny kolejne moje kodowanie

http://artu123.pl/projekty/iwanski/

Proszę o opinie i oceny .

[webber]

Widzę że powielasz błędy o których ludzie pisali Tobie w innych Twoich tematach

Spójrz na kod:

[HTML] pobierz, plaintext 
<div id="top">
     <img src="grafika/logotyp.gif" id="logotyp" alt="" /><!-- Koniec #logotyp -->
     <h1>iwanski<span class="literki">.com.pl</span></h1>
     <div id="krecha"></div><!-- Koniec #krecha -->
     <h2>Prywatna praktyka stomatologiczna<br /><span class="kolor">Marcin &nbsp;Iwanski &nbsp;Lekarz &nbsp;stomatolog</span></h2>
     <div id="kontakt">
          <img src="grafika/komorka.gif" id="komorka" alt="" />
          <div id="kontaktinfo">
               <p>ZADZWOŃ DO NAS</p>
               <span class="numer1">(+48)</span>
               <span class="numer">888 555 444</span>
          </div><!-- Koniec #kontaktinfo -->
     </div><!-- Koniec #kontakt -->
</div><!-- Koniec #top -->
[HTML] pobierz, plaintext 

Nie dawaj klas dla "literek" np. odwołaj się w .css do tego tak: div#top span { tu styl "literek" }

Nie używasz alt dla obrazków... dlaczego?

[HTML] pobierz, plaintext 
<ul id="menu">
        <li><h3><a href="#" title="Strona Główna">Strona główna</a></h3></li>
        <li><h3><a href="#" title="O nas">O nas</a></h3></li>
        <li><h3><a href="#" title="Certyfikaty">Certyfikaty</a></h3></li>
        <li><h3><a href="#" title="Kontakt">Kontakt</a></h3></li>
    </ul><!-- Koniec #menu -->
[HTML] pobierz, plaintext 

Nie lepiej zdeficniować wielkość czcionki odwołując się ul#menu li a { font-size: 1em; } ?

[HTML] pobierz, plaintext 
<li style="position:relative;width:5px;border:0px;background-color:#fff;margin-left:-1px;"></li>
[HTML] pobierz, plaintext 

tutaj mógłbyś dać po prostu class="cos" i zdefiniować styl w .css a nie inline.

[HTML] pobierz, plaintext 
<div class="przelacznik"><span class="sss">1</span> - <a href="#">2</a></div>
[HTML] pobierz, plaintext 

Tu odwołać się do span poprzez div#przelacznik span { styl }

To tylko przykłady.

Ogólnei tutaj na forum zwrócono mi uwagę, więc mogę powtórzyć, masz bardzo złe nazwy wszystkich warstw, powinny określać co się w nich znajduje, a nie to gdzie się znajdują
Po za tym nie sądzisz że czytelniej byłoby np. pisać .PrzerwaMiedzyPanelami a nie przerwamiedzypanelami?

Pozdrawiam

[Artu123]

"<li style="position:relative;width:5px;border:0px;background-color:#fff;margin-left:-1px;"></li>"

Dodałem class to nie działało .

[ano]

1. Po co <div id="tlo"></div><!-- Koniec #tlo --> ?
2. Pousuwaj z elementów głównego menu <h3>
3. To:

[HTML] pobierz, plaintext 
<div id="kontakt">
          <img src="grafika/komorka.gif" id="komorka" alt="" />
          <div id="kontaktinfo">
               <p>ZADZWOŃ DO NAS</p>
               <span class="numer1">(+48)</span>
               <span class="numer">888 555 444</span>
          </div><!-- Koniec #kontaktinfo -->
     </div><!-- Koniec #kontakt -->
</div><!-- Koniec #top -->
[HTML] pobierz, plaintext 

zrób używając jednego div'a (usuń div #kontaktinfo)
4. Do usunięcia:

[HTML] pobierz, plaintext 
<div id="krecha"></div><!-- Koniec #krecha -->
[HTML] pobierz, plaintext 

- wystarczy, że do <h1> dodasz border-bottom: (...)
5. Po co to:

[HTML] pobierz, plaintext 
<li style="position:relative;width:5px;border:0px;background-color:#fff;margin-left:-1px;"></li>
[HTML] pobierz, plaintext 

?
6. Te kreski: <div class="kreskalewa"></div> rób tak samo jak w punkcie 4. (border-bottom do <h3>)
7. To też do usunięcia: <div class="przerwamiedzypanelami"></div>
8. Tego też nie kumam: <div class="h3s"><h3>Lorem Ipsum is simply</h3></div> - po co kolejny div? samo h3 nie wystarczy?
9. <div class="float"> - to też do usunięcia (samo <a> ostylowane wystarczy!)
10. Po co <li style="width:76px;"> - w dolnym menu te width'y? Odległość pomiędzy kolejnymi <li> możesz przecież ustalić za pomocą padding-right!
11. <div id="kreskastopka"></div> - do wywalenia

Ogólnie: dużo za dużo <divów> - musisz uprościć kod, bo narazie jest divitis

tak na marginesie, wypadało by wykupić zdjęcia z istockphoto... http://artu123.pl/

Ten post edytował ano 29.03.2009, 13:08:11