Witam,

oddaje do oceny strone stworzoną dla firmy od zabezpieczeń itp

http://floox.webovers.org/juzstronanie/

prosze o konstruktywną krytyke i uwagi

Pozdrawiam

No dobra - jest ładne i się waliduje, ale co do kodu mam wiele zastrzeżeń.



Nie dość, że możnaby to (z użyciem CSS) napisać dużo prościej, to jeszcze niesemantyczne to jest.... alt="" powinien opisywać element, tym bardziej, że jest to związane z nawigacją (a nie np. zdjęcie). Co powiesz o niewidomych i botach? Skąd oni mają wiedzieć, że tam pisze "Kontakt" ?



CSS się kłania



I tu...



Tu też...



Takie rzeczy też można w CSS zrobić



No dajże człowieku spokój...

Podsumowanie:

Jest OK, ale jeszcze dużo pracy przed Tobą

buahahah, radex_p rozbawiłeś mnie jak nigdy, alt jest właśnie do opisu zdjęć a nie elementów nawigacji - do tego jest np title w <a>, a co do tego że można zrobić to w css zamiast w hatmlu się zgadzam
preloader obrazków w css zrobisz? a to Ciekawe

a uwagi od mojej skromnej osoby:
brak nagłówków, stosowanie <center> zamiast odpowiedniego ostylowania, menu powinno znaleźć się na liście, klasy w css można łączyć
zamiast kontrukcji w stylu

[HTML] pobierz, plaintext 
<div class="pierwszy"><p class="zawartosc">[test]></p></div>
[HTML] pobierz, plaintext 

powinno być raczej coś takiego

[HTML] pobierz, plaintext 
<p class="zawartosc pierwszy">[test]></p>
[HTML] pobierz, plaintext 

musisz (powinieneś) postarać się rozdzielić prezentację (css) od zawartości (html)

Tia, w zasadzie masz rację. Dla mnie alt jest do opisu generalnie obrazków, a opis elementu nawigacji na pewno będzie miał dla np. niewidomych większe znaczenie niż opis jakiegoś obrazka - faktem jest jednak, że title bardziej się do tego nadaje



Po co do obrazków na tak prostej stronie preloader? Nie można po prostu zrobić <img>a lub nadać background ?

wstawiłeś kod preloadera i napisałeś że da się w css więc pytam jak - bo bardzo by mi się taki bajer przydał Nigdzie nie napisałem że nie lepiej byłoby użyć obrazka jako background i pokazywać go fragmentarycznie bo faktycznie byłoby to w tym przypadku bardziej efektywne.
No i jeszcze mała uwaga odnośnie stopki wstawiasz tam dwa adresy do tej samej strony. Pytam się po co? Google traktuje takie coś raczej nie zbyt przychylnie (duplicated content) i nie jest to raczej zbyt wygodne - promować należy jedną domenę a jeśli posiadamy kilka i chcielibyśmy aby była pod nią ta sama zawartość to zwyczajnie robi się odpowiednie przekierowanie z serii http 30x i problem nie istnieje. Swoją drogą też takie podawanie 10 adresów z różnymi "końcówkami" do tej samej strony jest mało profesjonalnym podejściem ze strony firmy i powoduje że klient się gubi - bo na dobrą sprawę "gdyby to było to samo to po co by podawali dwa adresy?" - tak może pomyśleć osoba która wejdzie na stronę i zacznie klikać. zdziwi się widząc to samo i pewnie nie będzie zbyt dobrze kojarzyła takiej marki

dzięki za ocene

co do kodu to faktycznie mozna mieć zastrzezenia bo jakimś pro koderem to nie jestem

oddzielenie całkowicie html od cssa ciężko mi przychodzi, bo zawsze sobie cos tam dodaje w trakcie a potem zapominam usunąć czy tez przenieść do pliku .css

napewno uwagi wytknięte przez radex_p są słuszne
musze jeszcze poćwiczyć, popracować

pozdrawiam