Proszę o ocene:

www.citroenleasing.com

Proszę o krytykę grafiki, przejrzystości interfejsu, skryptów oraz htmla.

Pozdrawiam

Ogólnie stronka wygląda całkiem sympatycznie. Jednak:
- pod Operą się sypie (na 1280x800). Blok menu jest wyrównany do lewej natomiast blok treści do prawej krawędzi przeglądarki. Top jest na środku
- banner flashowy wygląda jak jakaś reklama pop-up, a to dlatego, że przekracza te poziome linie i jest jakby za blisko Business & Personal... Naprawdę w pierwszej chwili myślałem, że to reklama.
Jeśli natomaist chodzi o silnik:
- nie podobają mi się linki postaci http://www.citroenleasing.com/index.php/ci...-C5%20Hatchback - chodzi konkretnie o tą spację. Mógłbyś zamieniać je np. na _
Kod XHTML: jak dla mnie super Rażą tylko gdzieniegdzie css inline ale poza tym OK.
Ogólnie 8,5/10

Wyglad ladny, moze troszke ciasno
a co do kodu, w3 wywalil ponad 80 bledow ;>
no i osobiscie nie lubie flashy na stronie, ale ten jest maly i sie w miare komponuje, wiec ok
ogolnie ladna stronka

Już nie chciało się robić labeli dla pozostałych selectów w wyszukiwarce?

No panie

A zrobiłem tak: http://www.citroenleasing.com/index.php/citroen-';';

Active: obejrzyj strone gdy tlo w systemie operacyjnym nie jest biale.
Co prawda szary kolor MOGLBY pasowac, ale ostrzegam na wszelki wypadek.


Glowne menu (to na gorze) jest malo widoczne, a czerwnony napis ponizej odciaga uwage

"_Click here_ to register." a czemu nie po prostu "_Register_"

Manu na samym dole -- daj wieksze odstepy miedzy pozycjami lub jakies separatory

Mapa serwisu: a po co kazdy model, kazda wersja silnikowa wystepuje po kilka razy?


np. http://www.citroenleasing.com/index.php/le...oen-Picasso-VTX
"Contact Citroen Leasing to enquire about this offer or request a new quote using the form below"

tak patrzylem na ten formularz ponizej i nie wiedzialem po co on jest. Przydal by sie opis "Get quote" tak jak jest na form. obok "Send offer to a friend"


"View vehicle info" -- przydala by sie ikonka/text ze to jest w PDF; "delivery time" -- tez

a "Faq" to nie powinien byc "FAQ"?

1. przykro mi to mowic ale userów z opery jest ponizej 2% i tutaj niestety pieniądze poszły górą - supportu pod ten browser specjalnie nie bede robic - pieniądze.
2. Przemyślę to.
3. linki w tej postaci niestety zostaną bo nie przekazuję żadnych id, faktycznie mógłbym z tym kombinować ale znowu _ od strony seo nie pasują a - mogą zepsuć silnik jesli nazwa będzie zawierała oryginalnie - w treści. Z linkami google radzi sobie bardzo dobrze, więc myślę że zostanie.


Nie lubie validatora:) podobnie do pierwszej odpowiedzi - pieniądze - no i może brak mojego profesjonalizmu. Dzięki za ocenę.


Wstyd ze hoho. Umknął mi jeden escape string:) Fixed.


Nie rozumiem pierwszej części związanej z tłem. Menu nie jest ważne - ludzie szukają po kategoriach tak jak powinni. Click here faktycznie - fixed. Menu na dole jest dla googla, ale faktycznie troche szpeci - pomyśle. Mapa serwisu jest dla google, są tam wszystkie oferty. Get quote - ludzie wiedzą do czego - bo uzupełniają masowo. info o pdf dodam.

Dziękuje za opinie.

hmmm chyba nie tak trudno naprawić ten bug pod Operą no ale - jak chcesz

Jak dla mnie stronka bardzo ładna , trzeba popracować jeszcze z tego co widzę po wypowiedziach innych ale dla standardowego usera na 1280x760 strona prezentuje się ładnie a co najważniejsze profesjonalnie.

witam

jak otworzysz pustą stronę w przeglądarce to tło tej pustej strony nie zawsze jest białe - zależy to od systemu/stylu dlatego właśnie zawsze trzeba podawać tło dla body (nawet jak ma być białe)

pozdrawiam

no ale wlasnie fragment http://citroensolutions.com/main.css:

O_o, jak chcesz


chodzilo o:

Czyli to tlo mialo byc takie szare? Szaremu dawałem 1%


A nie lepiej zrobic to na poziomie dostepu do bazy danych, np. przy podpinaniu wartosci pod zapytanie automatycznie escapowac stringi?

Żeby strona wyświetlała się poprawnie pod każdą przeglądarką (O, IE, FF) wystarczy tylko dopisać do styli #content:

[XML] pobierz, plaintext 
width: 776px;
      margin: 0 auto;
[XML] pobierz, plaintext 

tj.:

[XML] pobierz, plaintext 
#content{
      background-image: url('left_men.gif');
      background-repeat: repeat-y;
      background-position: top left;
      width: 776px;
      margin: 0 auto;
}
[XML] pobierz, plaintext