Zaczynam przygode z kodowaniem,
prosze o ocene tego co zrobiłem, prosty bo prosty lay ale..
spayk.ugu.pl
Pełna wersja: [HTML][CSS]Ocena kodu
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:
7. Stosujesz za dużo ID.
Reasumując, dużo nauki przed Tobą.
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:
7. Stosujesz za dużo ID.
Reasumując, dużo nauki przed Tobą.
te puste div robią tylko tło ;/
multi background jakoś mi nie funka
menu w div bo uciekało pod logo
multi background jakoś mi nie funka
menu w div bo uciekało pod logo
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:
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:
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.
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>
...
<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 */
}
/* 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.
o co chodzi z zamykaniem znaczników ?
bo KED mi je sam zamyka
bo KED mi je sam zamyka
Chodzi o to, że coś takiego:
powinno wyglądać tak:
powinno wyglądać tak:
Na przykładzie powyżej może tobie wyjaśnię:
Prawidłowo:
Div jako w twoim przypadku jest blokiem,kontenerem w którym umieszczasz elementy-u ciebie jest to tag <a>
tag więc robisz tak:
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.
Prawidłowo:
Div jako w twoim przypadku jest blokiem,kontenerem w którym umieszczasz elementy-u ciebie jest to tag <a>
tag więc robisz tak:
Cytat
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>
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.
Cytat(spayk @ 6.11.2011, 19:42:06 ) 
o co chodzi z zamykaniem znaczników ?
bo KED mi je sam zamyka
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.
uff to przez nie dopatrzenie 
a co do tych div to oprócz photo jest dobrze reszta czy tam tez class
a co do tych div to oprócz photo jest dobrze reszta czy tam tez class
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. :-)
Ps. Taka wspominka, kiedyś kodowałem stronę od firmy x i prosili by w ogóle nie korzystać z atrybutu id. :-)
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.