[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.

[pedro84]

1. Czemu nie HTML5? Trzeba iść do przodu.
2. Po co menu jest wsadzone w diva?
3. <br /> służy do czegoś innego, nie odstępów między elementami.
4. Powtarzasz id w galerii: id="photo"
5. Puste divy
6. Źle zamykasz znaczniki:

[HTML] pobierz, plaintext 
<div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>
[HTML] pobierz, plaintext 

7. Stosujesz za dużo ID.

Reasumując, dużo nauki przed Tobą.

[spayk]

te puste div robią tylko tło ;/
multi background jakoś mi nie funka
menu w div bo uciekało pod logo

[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

[spayk]

o co chodzi z zamykaniem znaczników ?
bo KED mi je sam zamyka

[Korab]

Chodzi o to, że coś takiego:

[HTML] pobierz, plaintext 
<div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>
[HTML] pobierz, plaintext 

powinno wyglądać tak:

[HTML] pobierz, plaintext 
<div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii" /></a></div>
[HTML] pobierz, plaintext 

[Niktoś]

Na przykładzie powyżej może tobie wyjaśnię:

[HTML] pobierz, plaintext 
<div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>
[HTML] pobierz, plaintext 

Prawidłowo:

[HTML] pobierz, plaintext 
<div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii" /></a></div>
[HTML] pobierz, plaintext 

Div jako w twoim przypadku jest blokiem,kontenerem w którym umieszczasz elementy-u ciebie jest to tag <a>
tag więc robisz tak:

1.<div> </div> i wstawiasz do niego tag <a>:
2.<div><a href="#"></a></div>-do tagu<a> wstawiasz odnośnik w postaci tagu <img>.
3.<div><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii" /></a></div>

ze względu że doctype jest xhtml(W HTML5 także można domykać w ten sposób ) to tagi(nie wszystkie,musisz sam obczaić które) możesz domykać tak <img /> , jakby był to doctype html 4.01 to ten tag musiałbyś zamknąć w ten sposób <img>.

Jak edytor robi ci taką sieczkę,to może przestań mu ufać,albo zmień na inny.

Heh 15min zajęło pisanie tego posta ,bo poprzednik był szybszy.

Ten post edytował Niktoś 6.11.2011, 22:08:38

[92nasti]

o co chodzi z zamykaniem znaczników ?
bo KED mi je sam zamyka

Widzę że poprzednicy nie potrafią zamienić tego w kilka prostych słów. Chodzi o to aby zamykać znacznik w odwrotnej kolejności do ich otwierania.

Ten post edytował 92nasti 6.11.2011, 20:55:45

[spayk]

uff to przez nie dopatrzenie

a co do tych div to oprócz photo jest dobrze reszta czy tam tez class

[ShadowD]

Prosta zasada, jeśli wezmę Twój kod nie mogę w nim znaleźć dwóch elementów z jednym id, id się stosuje do elementów typu główne logo, jakiś główny baner, podział strony na jakieś większe części gdzie każda z nich jest inna, a klasy do wszystkiego innego lepiej jeśli się nie wie z czego korzystać użyć za dużo class niż id - choć nie pochwalam takich mechanizmów.

Ps. Taka wspominka, kiedyś kodowałem stronę od firmy x i prosili by w ogóle nie korzystać z atrybutu id. :-)