Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www] kodowanie
Forum PHP.pl > Inne > Oceny
Artu123
Witam!

Mam do oceny tylko moje kodowanie . Lay nie ja robiłem więc się nie czepiajcie . Co poprawić itp.. ? Mam pytanie, ile w tej chwili na moje umiejętności mogę brać za kodowanie pod xhtml/css ? Na ile oceniacie .

Edit://

Tutaj link do kodowania:

http://test.grafa.netarteria.eu/projekty/umaping/
kantek
Kod ładny i czytelny. Nie podoba mi się tylko, że masz dużo definicji stylu w kodzie html,
chodzi mi o <div style="...."> od tego jest css.

Takie coś mi się nie podoba:
  1. <div class="clear" style="height:4px;"></div>
  2. <div style="text-align:center;"><input type="reset" value="Resetuj" class="submit" />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
  3. <input type="submit" value="Zaloguj" class="submit" /></div>


div dajesz pusty który ma 4px wysokości - możesz to zrobić marginem innego divu, a czy
ie6 się nie wykrzacza na tym bez overflow:hidden (nie mam teraz jak sprawdzić) questionmark.gif

i te spacje to samo

Ogólnie : 9/10 (też robię wcięcia w css smile.gif )

Pozdrawiam
Kildyt
Tytuł jest jednym z najważniejszych rzeczy w pozycjonowaniu. Ten nic nie mówi.
Puste div'y są brzydkie. Po co podajesz alt'y jeżeli pozostawiasz je puste? Zielone światełko w walidatorze za wszelką cenę to bezsens.
Jak już kantek napisał: za dużo stylów w html-u.
Zacznij używać label-ów.
  1. &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
Bardzo źle to wygląda.
Używasz h3, gdy nie ma h2.
Jeżeli w całym znaczniku używasz wielkich liter to zastosowałbym polecenie w css-ie, żeby się nie męczyć z Caps Lock'iem. smile.gif

Ogólnie za dużo div-ów. Całkiem fajnie. winksmiley.jpg
webber
"style="padding-top:1px;""

Styl inline.

"id="NadTop", class="NadTopMenu" - już i mi zwrócono uwagę. Nie używaj takich nazw. Użyj "menu" "subMenu" "headerMenu".

<b>Sprite</b> -> <strong> winksmiley.jpg

Kod logowania, formularza jest TRAGICZNY. Google.pl -> "clean forms xhtml" "forms web standards" winksmiley.jpg

Czemu nie używasz atrybutu "alt"?

  1. <div class="PanelPadding">
  2.  
  3. <ul class="NawigacjaPanel">
  4.  
  5. <li><a href="#">Strona Główna</a></li>
  6. <li><a href="#">Strona Główna</a></li>
  7.  
  8. <li><a href="#">Strona Główna</a></li>
  9. <li><a href="#">Strona Główna</a></li>
  10. <li><a href="#">Strona Główna</a></li>
  11. <li><a href="#">Strona Główna</a></li>
  12.  
  13. </ul>
  14.  
  15. <div class="clear"></div>
  16.  
  17. </div><!-- Koniec .PanelPadding -->


Tu zamiast dawać klasę do UL mogłeś się odwołać do tego poprzez div.PanelPadding ul { }


Na IE 8 rozjeżdża się Tobie menu na topie.

Ile możesz brać? Na zlecenia.przez.net jakieś 200 od strony - tak myślę. Profesjonalna robota to to jeszcze nie jest winksmiley.jpg

6/10

Cytat
Tytuł jest jednym z najważniejszych rzeczy w pozycjonowaniu. Ten nic nie mówi.


Toć to nie jego sprawa jaki będzie tytuł strony!

Cytat
Jak już kantek napisał: za dużo stylów w html-u.


sęk w tym że wcale ich tam nie powinno być

Cytat
Puste div'y są brzydkie.


Choć skuteczne... :/
Artu123
Cytat(webber @ 19.04.2009, 19:20:03 ) *
"style="padding-top:1px;""

Styl inline.

"id="NadTop", class="NadTopMenu" - już i mi zwrócono uwagę. Nie używaj takich nazw. Użyj "menu" "subMenu" "headerMenu".

<b>Sprite</b> -> <strong> winksmiley.jpg

Kod logowania, formularza jest TRAGICZNY. Google.pl -> "clean forms xhtml" "forms web standards" winksmiley.jpg

Czemu nie używasz atrybutu "alt"?

  1. <div class="PanelPadding">
  2.  
  3. <ul class="NawigacjaPanel">
  4.  
  5. <li><a href="#">Strona Główna</a></li>
  6. <li><a href="#">Strona Główna</a></li>
  7.  
  8. <li><a href="#">Strona Główna</a></li>
  9. <li><a href="#">Strona Główna</a></li>
  10. <li><a href="#">Strona Główna</a></li>
  11. <li><a href="#">Strona Główna</a></li>
  12.  
  13. </ul>
  14.  
  15. <div class="clear"></div>
  16.  
  17. </div><!-- Koniec .PanelPadding -->


Tu zamiast dawać klasę do UL mogłeś się odwołać do tego poprzez div.PanelPadding ul { }


Na IE 8 rozjeżdża się Tobie menu na topie.

Ile możesz brać? Na zlecenia.przez.net jakieś 200 od strony - tak myślę. Profesjonalna robota to to jeszcze nie jest winksmiley.jpg

6/10



Toć to nie jego sprawa jaki będzie tytuł strony!



sęk w tym że wcale ich tam nie powinno być



Choć skuteczne... :/


Za takie np. kodowanie 200zł ? sciana.gif

Ja za to wziąłem 50zł aaevil.gif haha.gif
mike
Cytat(Artu123 @ 19.04.2009, 19:27:42 ) *
Ja za to wziąłem 50zł aaevil.gif haha.gif
Mało wziąłeś. Bo tyle kosztuje amatorszczyzna.
To co dałeś tu do oceny niewiele różni się od wszystkich poprzednich "prac". Nie potrafisz dokonać prostego postępu.
webber
Więc nie masz szacunku do własnej pracy.

No chyba że pociąłeś to w godzinę winksmiley.jpg

IT to nie branża w której chodzi o to by zarobić sobie na piwko, no chyba że traktujesz to jako hobby ;p

Staraj się wyeliminować używanie stylów wewnątrz kodu html, staraj się jak najmniej używać class smile.gif
Artu123
Ja robię tak:

dodaje w css klasę clear i później w html dodaj div z klasa clear i dodaje do tego style i ustalam height tylko ile chce . Robie style inline no, ale nie tworzę nowych klas . Co lepsze ?

Tworzyć nową klasę, czy dodawać do style wysokość ?
webber
Cytat(Artu123 @ 19.04.2009, 19:34:31 ) *
Ja robię tak:

dodaje w css klasę clear i później w html dodaj div z klasa clear i dodaje do tego style i ustalam height tylko ile chce . Robie style inline no, ale nie tworzę nowych klas . Co lepsze ?

Tworzyć nową klasę, czy dodawać do style wysokość ?


Aaa tu nie chodzi o clear, tu chodzi o typowe marginy itd.

Co do Twojego pytania, nie wiem. Ale myślę że klasę. Wtedy zamiast zmieniać w wielu miejscach to i szukać po całym dokumencie otwierasz 1 plik .css masz wydzieloną strefę z wszystkimi "clear`ami" i sobie je tam edytujesz.
Kildyt
Cytat(mike @ 19.04.2009, 19:31:40 ) *
Mało wziąłeś. Bo tyle kosztuje amatorszczyzna.
Może z allegro wziął zlecenie? tongue.gif

Cytat(webber @ 19.04.2009, 19:20:03 ) *
<b>Sprite</b> -> <strong> winksmiley.jpg
Tylko nie popadajmy w paranoję. Nie wszystkie "b" powinny być strong'iem. Strong do zaznaczenia ważniejszej treści, a b do designu.

Cytat(webber @ 19.04.2009, 19:20:03 ) *
Toć to nie jego sprawa jaki będzie tytuł strony!
No tak, ale dobre praktyki trzeba pokazywać na każdym kroku. smile.gif

Cytat(webber @ 19.04.2009, 19:20:03 ) *
sęk w tym że wcale ich tam nie powinno być
Ja wstawiam style gdy dany element występuje dość rzadko i jest ich dosyć mało. Nie ma sensu zaśmiecać nim css-a.

Cytat(webber @ 19.04.2009, 19:20:03 ) *
Choć skuteczne... :/
No tak. Dzisiaj nawet zauważyłem, że sam takowych używam. Czasami niestety brakuje odpowiedniego znacznika. Postaram się sam siebie zmobilizować do unikania tego typu sytuacji. smile.gif
webber
Cytat
Ja wstawiam style gdy dany element występuje dość rzadko i jest ich dosyć mało. Nie ma sensu zaśmiecać nim css-a.


Hm, ale zauważ że jednak masz łatwiejszy dostęp do tego, nie nazwałbym tego zaśmiecaniem .css

Wtedy jak ktoś ma edytować cokolwiek po Tobie otwiera sobie plik .css, uruchamia Web Developer do FF i sobie edytuje wszystko jak mu się podoba jedynie za pomocą .css :]

Ale poczekajmy na opinię kogoś bardziej zaawansowanego, czy stosować styl inline w niektórych przypadkach czy tworzyć klasę smile.gif
Artu123
w Alt mam wpisywać co to za obrazek ? np. Logo ?
webber
Cytat(Artu123 @ 20.04.2009, 06:52:46 ) *
w Alt mam wpisywać co to za obrazek ? np. Logo ?


http://www.queensu.ca/www/wsaguide/development/alt.html

To Tobie pomoże smile.gif
drake88
Według standardów Twoje kodowanie jest jak najbardziej zgodne z xHTML.
piotrooo89
Cytat
Według standardów Twoje kodowanie jest jak najbardziej zgodne z xHTML.


co nie znaczy że jest semantyczne. ~mike kiedyś powiedział że nawet jeśli gówno opakuje się w super paczke dalej będzie gównem (czy coś koło tego).

PS. bez aluzji.
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.