Kodowanie xHtml Opcje

[Artu123]

http://test.grafa.netarteria.eu/projekty/shoutbox/

Witam !

Pociąłem ten lay i zakodowałem go pod xhtml . Zapraszam do oceny kodu .

[Crozin]

1. Przerzuć się na angielskie nazewnictwo.

2. [HTML] pobierz, plaintext 
   <h1><a href="#" title="GamesPlay.pl - Shoutbox"></a></h1>
   [HTML] pobierz, plaintext 

   Chyba zapomniałeś o treści dla A, a link jako # jest złym rozwiązaniem. Daj jako link do głównej.

3. [HTML] pobierz, plaintext 
   <p><strong>Chatują</strong> :</p>
   [HTML] pobierz, plaintext 

   A co jest takiego w tym ważnego, że dajesz to aż w STRONG?

4. [HTML] pobierz, plaintext 
   <div id="WymienKtoChat">
    
   			<a href="#" title="Artu123">Artu123</a> , <a href="#" title="Artu123">Artu123</a> , <a href="#" title="Artu123">Artu123</a> , <a href="#" title="Artu123">Artu123</a>
    
   		</div>
   [HTML] pobierz, plaintext 

   To jest przecież lista, więc wyświetl to jako listę.
5. Przed nadaniem nazwy identyfikatora czy klasy zastanów się czy ma ona jakiś sens: WymienKtoChat, WymienKtoChat1?
6. Cierpisz na divitis i classists
7. Kod HTML powinienen reprezentować strukturę danych, a nie jej wygląd - więc pozbądź się jakiś "ContentLewa" (ponglisha też)
8. Po co co chwile jakiś DIV z clearem?
9. Staraj się nie stosować atrybutu style, chyba że to konieczne - tutaj nie jest.
10. line-height:0px;font-size:0px; co to ma być?

11. [HTML] pobierz, plaintext 
    <img src="images/AvatarTop.gif" alt="[Avatar - Border]" /><br />
    				<img src="images/Avatar.gif" class="AVATAREK" width="59px" height="52px" title="Artu123 - Avatar" alt="[Avatar]" /><br />
    				<img src="images/AvatarDown.gif" alt="[Avatar - Border]" />
    [HTML] pobierz, plaintext 

    Obraz wstawiany przez IMG to część "contentu", a nie dekoracji - więc wywal te "bordery".

    Skoro dajesz klasę to po co width i height pisane inline?

    Akurat dobrze, że to podał - strona nie będzie skakać przy ładowaniu. Jedynie niepotrzebnie dodał jednostkę (px).

12. [HTML] pobierz, plaintext 
    <div class="ShoutboxTrescNick">
     
    					<a href="#">Artu123</a>
     
    				</div>
     
    				<div class="ShoutboxTrescData">
     
    					07.05/20:23
     
    				</div>
    [HTML] pobierz, plaintext 

    HTML wychodzi do Ciebie z dziesiątkami elementów, a Ty się uparłeś tych DIVów. O CITE, BLOCKQUOTE, P i całej masie innych nie słyszałeś?

13. [HTML] pobierz, plaintext 
    <!-- Koniec Post -->
    [HTML] pobierz, plaintext 

    Polski jest językim, w którym niektóre części mowy/zdania się odmieniają. Wiem, że się czepiam.

14. [HTML] pobierz, plaintext 
    <textarea rows="0" cols="0"></textarea>
    [HTML] pobierz, plaintext 

    Poza tym, że zapomniałeś o atrybucie name, to zapomniałeś jeszcze o formularzu. No i gdzie jakiś opis (LABEL) na co jest to pole?

ps. jak widzę coś takiego ->
...
to jestem przerażony... gdzieś widziałem tutorial gdzie pisali że TRZEBA czegoś takiego używać - dla mnie głupota i śmietnik!

Jak gdzieś napisano, że tak trzeba to rzeczywiście głupota, ale dlaczego uważasz resetowanie domyślnych stylów przeglądarki za głupotę? Pozwala to uniknąć wielu niepotrzebnych nieścisłości.

1. Kod CSS jest kompletnie nieczytelny
2. Przy zerze nie dawaj jednostek
3. A... nie sprawdzam, bo nie chce mi się tego rozszyfrowywać...