[HTML][CSS]Ocena kodu Opcje

[spayk]

Zaczynam przygode z kodowaniem,
prosze o ocene tego co zrobiłem, prosty bo prosty lay ale..
spayk.ugu.pl

Powód edycji: [Daiquiri]: Przenoszę do odpowiedniego działu.

[Speedy]

id jest unikalnym atrybutem. Nie możesz go stosować do wielu elementów. Konkretną nazwę "id" może mieć tylko jeden element w całym kodzie.
Jeżeli zacząłbyś korzystać z jQuery w Twoim kodzie i odwoływał do struktury DOM po "id", pewnie pojawiłyby się błędy, bo nie wiadomo będzie, o który "id" chodzi.
Masz w swoim kodzie np. coś takiego:

Kod

...
<div id="gallery">
   <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>
   <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>
   <div id="photo"><a href="#" ><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>
</div>
...

Jeżeli chcesz użyć tych samych właściwości dla wielu elementów, to użyj atrybutu "class" zamiast "id".
Możesz w ogóle pozbyć się tych atrybutów i zrobić w swoim arkuszu CSS coś takiego:

Kod

#gallery div {
   /* właściości */
}

Jeżeli wszystkie div-y w kontenerze #gallery będą musiały posiadać te same cechy, to w takim wypadku możesz w ogóle pominąć atrybuty "class" oraz "id". Dzięki temu automatycznie zmniejszasz objętość kodu HTML.

Dodatkowo dochodzi problem złego zamykania znaczników, o którym wspomniał przedmówca.

Tyle rzuciło mi się na pierwszy rzut oka. Może potem dokładniej to przeanalizuję i coś napiszę w wolnej chwili.

Ten post edytował Speedy 6.11.2011, 19:39:06