 [www] G-notice |
| [www] G-notice |
Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 34 Pomógł: 0 Dołączył: 10.06.2008 Skąd: Katowice Ostrzeżenie: (0%) 
 |
Witam,
oddaje do oceny strone stworzoną dla firmy od zabezpieczeń itp http://floox.webovers.org/juzstronanie/ prosze o konstruktywną krytyke i uwagi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Pozdrawiam Ten post edytował FlooX 15.06.2008, 16:30:59 |
 |
 
|
 |
|
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 1 657 Pomógł: 125 Dołączył: 29.04.2006 Ostrzeżenie: (0%)
|
No dobra - jest ładne i się waliduje, ale co do kodu mam wiele zastrzeżeń.
Kod <div><a href="index.php?id=kontakt"> <img src="jpg/kontaktBtn.jpg" name="kontaktBtn" border="0" id="kontaktBtn" class="menu" alt="" /></a> </div> 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" ? Kod <div id="przerwa" ><img src="jpg/rozdzielaczTextu.jpg" alt="" /></div> CSS się kłania Kod <div id="stopka"><img src="jpg/stopka.jpg" alt="" /><br /> I tu... Kod <p class="zawartosc" align="left"> Tu też... Kod <body onload="MM_preloadImages('jpg/ofirmieBtnZaz.jpg','jpg/uslugiBtnZaz.jpg','jpg/downloadBtnZaz.jpg','jpg/kontaktBtnZaz.jpg','jpg/uslugi/sysBtn2.jpg','jpg/uslugi/cctvBtn2.jpg','jpg/uslugi/domofBtn2.jpg','jpg/uslugi/kompBtn2.jpg','jpg/uslugi/siecBtn2.jpg')"> Takie rzeczy też można w CSS zrobić Kod <div style="width:722px; height:310px"> <div id="lewa" align="center"><img src="jpg/lewa.jpg" border="0" usemap="#Map" alt="" /> <map name="Map" id="Map"><area shape="rect" coords="157,179,163,184" href="http://velga.republika.pl/irlandia/" target="_blank" alt="" /> </map></div> No dajże człowieku spokój... Podsumowanie: Jest OK, ale jeszcze dużo pracy przed Tobą |
|
|
|
|
Post
#3
|
|
|
Grupa: Zarejestrowani Postów: 1 470 Pomógł: 75 Dołączył: 21.09.2005 Skąd: że znowu Ostrzeżenie: (0%)
|
Cytat 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" ? 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ę zgadzamCytat Takie rzeczy też można w CSS zrobić preloader obrazków w css zrobisz? a to Ciekawea 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 powinno być raczej coś takiego musisz (powinieneś) postarać się rozdzielić prezentację (css) od zawartości (html) |
|
|
|
|
Post
#4
|
|
|
Grupa: Zarejestrowani Postów: 1 657 Pomógł: 125 Dołączył: 29.04.2006 Ostrzeżenie: (0%)
|
Cytat 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> 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 (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Cytat preloader obrazków w css zrobisz? a to Ciekawe Po co do obrazków na tak prostej stronie preloader? Nie można po prostu zrobić <img>a lub nadać background ? |
|
|
|
|
Post
#5
|
|
|
Grupa: Zarejestrowani Postów: 1 470 Pomógł: 75 Dołączył: 21.09.2005 Skąd: że znowu Ostrzeżenie: (0%)
|
Cytat 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ł (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) 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 |
|
|
|
|
Post
#6
|
|
|
Grupa: Zarejestrowani Postów: 34 Pomógł: 0 Dołączył: 10.06.2008 Skąd: Katowice Ostrzeżenie: (0%)
|
dzięki za ocene (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) musze jeszcze poćwiczyć, popracować pozdrawiam |
|
|
|
 |
|
Aktualny czas: 24.08.2025 - 10:48 |
