[www]Strona auto serwisu Opcje

[piotrooo89]

proszę o ocenę http://www.auto-serwis.trzebinia.com/beta/index.php i konstruktywną krytykę.

grafikiem nie jestem więc bądźcie delikatni (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[chudykrzys]

Na plus:

• Przejrzysta grafika, bardzo prosta, aż za prosta chwilami (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
• dobry kod HTML + CSS, chociaż masz jeden drobny błąd (sprawdź stronę w walidatorze)

Na minus:

• Brak przyjaznych urli,
• Każda podstrona powinna mieć inne znaczniki meta, u Ciebie wszystkie są takie same, na dodatek w keywords masz literówkę (smochody).
• w formularzu kontaktowym klikam wyślij bez wypełnienia żadnej treści. Brak żadnego komunikatu o błędzie.
• po kliknięciu design by Twoja strona powinna otworzyć się na nowej stronie.
• zamiast <br style="clear: both;"/> poprawniej jest stosować nowocześniejszy clearfix
• byłoby poprawniej gdybyś połączył wszystkie pliki css w jeden. A javascript przeniósł przed znacznik zamykający </body>
• odstępy w tekście powinieneś uzyskiwać poprzez css i marginesy a nie wstawienia <br/>.
• zmieniasz rozmiary obrazków w html (<img height="100" width="100" alt="galeria" src="gallery/diagnoskop1.jpg" class="mini"/>) przez co powstaje pixeloza np. na stronie wyposażenie, diagnostyka silnika. Powinieneś mieć dwa obrazki jeden normalny drugi miniaturkę przygotowaną w programie graficznym.

Bardzo się czepiam, większość z tych rad jest przesadzona na tego typu serwis.
Może spróbuj poszukać jednak jakiś inny nawet darmowy layout (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[JoShiMa]

[*]dobry kod HTML + CSS, chociaż masz jeden drobny błąd (sprawdź stronę w walidatorze)

Żarty sobie stroisz? Co w tym kodzie jest dobrego? Mieszanie stylów z znacznikami typu <center> czy <strong>?
Albo na przykład takie konstrukcje:

[HTML] pobierz, plaintext 
<br />
<br />
[HTML] pobierz, plaintext 

A w ogóle jaka idea przyświecała nadawaniu znacznikowi bodu atrybutu id?

Takie coś:

[HTML] pobierz, plaintext 
<span class="uslugi_tyt">Naprawa zawieszenia</span>
[HTML] pobierz, plaintext 

albo takie:

[HTML] pobierz, plaintext 
<p class="kategoria">Geometria</p>
[HTML] pobierz, plaintext 

To się robi za pomocą znaczników <hx> Tymczasem na Twojej stronie nie ma żadnego znacznika <hx>


Wygląd kiepski. Grafika nie jest skromna ale rzec by można siermiężna. Typografia tragiczna. Dobór kolorów również zostawia wiele do życzenia.

Jednym słowem jeszcze wiele wiele pracy przed tobą.

[chudykrzys]

Żarty sobie stroisz? Co w tym kodzie jest dobrego? Mieszanie stylów z znacznikami typu <center> czy <strong>?
Albo na przykład takie konstrukcje:

[HTML] pobierz, plaintext 
<br />
<br />
[HTML] pobierz, plaintext 

Miałem na myśli to, że w kodzie nie ma chociaż tabelek (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[JoShiMa]

Miałem na myśli to, że w kodzie nie ma chociaż tabelek (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Jest to jakies minimum, ale do przyzwoitego kodu to jeszcze baaardzo daleko.