Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [HTML][CSS]Ocena kodu
spayk
post
Post #1





Grupa: Zarejestrowani
Postów: 364
Pomógł: 3
Dołączył: 29.11.2008

Ostrzeżenie: (0%)
-----


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.
Go to the top of the page
+Quote Post
pedro84
post
Post #2





Grupa: Nieautoryzowani
Postów: 2 249
Pomógł: 305
Dołączył: 2.10.2006

Ostrzeżenie: (0%)
-----


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:
  1. <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>

7. Stosujesz za dużo ID.

Reasumując, dużo nauki przed Tobą.
Go to the top of the page
+Quote Post
spayk
post
Post #3





Grupa: Zarejestrowani
Postów: 364
Pomógł: 3
Dołączył: 29.11.2008

Ostrzeżenie: (0%)
-----


te puste div robią tylko tło ;/
multi background jakoś mi nie funka
menu w div bo uciekało pod logo
Go to the top of the page
+Quote Post
Speedy
post
Post #4





Grupa: Zarejestrowani
Postów: 651
Pomógł: 28
Dołączył: 4.12.2004

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post
spayk
post
Post #5





Grupa: Zarejestrowani
Postów: 364
Pomógł: 3
Dołączył: 29.11.2008

Ostrzeżenie: (0%)
-----


o co chodzi z zamykaniem znaczników ?
bo KED mi je sam zamyka
Go to the top of the page
+Quote Post
Korab
post
Post #6





Grupa: Zarejestrowani
Postów: 202
Pomógł: 36
Dołączył: 10.06.2011
Skąd: Dokąd

Ostrzeżenie: (0%)
-----


Chodzi o to, że coś takiego:
  1. <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>

powinno wyglądać tak:
  1. <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii" /></a></div>
Go to the top of the page
+Quote Post
Niktoś
post
Post #7





Grupa: Zarejestrowani
Postów: 1 195
Pomógł: 109
Dołączył: 3.11.2011

Ostrzeżenie: (10%)
X----


Na przykładzie powyżej może tobie wyjaśnię:
  1. <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii"></div></a>

Prawidłowo:
  1. <div id="photo"><a href="#"><img src="photo/photo.jpg" alt="Przejdź do galerii" /></a></div>

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>


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 (IMG:style_emoticons/default/ohmy.gif) ,bo poprzednik był szybszy.

Ten post edytował Niktoś 6.11.2011, 22:08:38
Go to the top of the page
+Quote Post
92nasti
post
Post #8





Grupa: Zarejestrowani
Postów: 80
Pomógł: 1
Dołączył: 22.01.2011

Ostrzeżenie: (0%)
-----


Cytat(spayk @ 6.11.2011, 19:42:06 ) *
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
Go to the top of the page
+Quote Post
spayk
post
Post #9





Grupa: Zarejestrowani
Postów: 364
Pomógł: 3
Dołączył: 29.11.2008

Ostrzeżenie: (0%)
-----


uff to przez nie dopatrzenie (IMG:style_emoticons/default/haha.gif)

a co do tych div to oprócz photo jest dobrze reszta czy tam tez class
Go to the top of the page
+Quote Post
ShadowD
post
Post #10





Grupa: Zarejestrowani
Postów: 1 333
Pomógł: 137
Dołączył: 25.03.2008
Skąd: jesteś??

Ostrzeżenie: (0%)
-----


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. :-)
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 25.08.2025 - 14:26