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

grafikiem nie jestem więc bądźcie delikatni

Szczerze ? Nie ma czego oceniać, bo co tam zrobiłeś ? Grafika jest fatalna ale to chyba sam wiesz. Do tego taka mała stronka a i tak się nie validuje - chociaż to popraw Do formularza przydałoby się jakieś zabezpieczenie, oraz jeśli wpiszemy zły adres tj. http://www.auto-serwis.trzebinia.com/beta/...?page=blebleble, to żeby nie było pustej strony tylko info jakieś o błędzie, lub przekierowanie na główną. Generalnie podtrzymuję swoje zdanie z początku - nie ma co oceniać, więc powstrzymam się od wydawania niskich ocen

Na plus:

• Przejrzysta grafika, bardzo prosta, aż za prosta chwilami
• 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

Ż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ą.

Miałem na myśli to, że w kodzie nie ma chociaż tabelek

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

Ja przyłączam się do oceny moich poprzedników w tym temacie. Dodam jeszcze coś od siebie.



Hmmm... delikatnie mówiąc takie teskty na pierwszej stronie można umieszczać. Ale nigdy nie same. Pierwsza strona (zwłaszcza firmowa) powinna stanowić choć krótkie przybliżenie działalności. Wywodzić szczegółowo możesz się w podstronie oferta.

Tak przynajmniej wyglądają te mniej prymitywne "firmówki".

Dobrą ofertą zachęcisz ludzi, żeby w ogóle choć zechcieli przejrzeć podstrony Twojego serwisu. Swoją drogą zainstaluj statystyki i trzymaj rękę na pulsie.

Zmień pierwszą stronę i porównuj czas jaki spędzają Twoi klienci na Twojej stronie i czy wchodzą na podstrony. Nikt nie każe Ci zostawiać cały czas takiej samej. Grunt to nad tym pracować.

Grafika...

Rozmyty top jest zupełnie nie na miejscu... Zrób go wyraźnie. Auto serwis jeszcze bez większych problemów przeczytałem, ale Trzebiny musiałem się dopatrywać zbyt długo. Klient nie lubi się przypatrywać co jest napisane.


Pozdrawiam,
Kamil Jura