![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 5 Pomógł: 0 Dołączył: 7.03.2009 Ostrzeżenie: (0%) ![]() ![]() |
Witam, prosiłbym Was o ocenę kodowania i ogólnie stronki. Być może pewne elementy można by było lepiej rozwiązać. Oto adres http://www.metro-plast.pl/test/
Dziękuje i pozdrawiam |
|
|
![]() |
![]()
Post
#2
|
|
Newsman Grupa: Moderatorzy Postów: 2 033 Pomógł: 290 Dołączył: 21.12.2007 Skąd: Łódź ![]() |
Generalnie mnie się bardzo podoba, bardzo estetycznie wykonana i czytelna strona, żadnych zbędnych bajerów, przyjemna, ze smakiem dobrana kolorystyka.
Walidator też pochwalił (IMG:style_emoticons/default/smile.gif) ( http://validator.w3.org/check?uri=http%3A%...ine&group=0 ) Przyczepiłbym się do zbyt ubogiego top-a (tego z logiem), tylko nad tym bym Ci radził jeszcze popracować. Aha, i w elementach głównego menu brakuje polskich znaczków. |
|
|
![]()
Post
#3
|
|
Grupa: Zarejestrowani Postów: 2 885 Pomógł: 463 Dołączył: 3.10.2009 Skąd: Wrocław Ostrzeżenie: (0%) ![]() ![]() |
Ogólnie ok, ale:
- gdzie są polskie znaki? - subskrypcja nie działa (nie mogę wpisać adresu e-mail, ale rozumiem, że to jeszcze nie jest gotowe) - po kliknięciu w zdjęcie opakowania chciałbym zobaczyć powiększoną fotkę lub opis produktu, tymczasem nic się nie dzieje - dlaczego urządzenia pakujące otwierają się w nowej zakładce? chyba lepiej w tym samym oknie + jakiś link powrotny - teksty o firmie powinny być justowane tak jak jest w urządzeniach pakujących, a nie tylko wyrównywane do lewego akapitu - w rozdzielczości na której pracuję (1680 x 1050) strona sprawia wrażenie zwężonej z bardzo małą ilością miejsca na tekst właściwy (gdyby połączyć oba boczne marginesy to zabrałyby ok. połowę miejsca wszerz - nieprawidłowy link do pobrania katalogu - niepoprawny stylistycznie fragment tekstu na stronie głównej: "(...) W najbliższym czasie powstanie nasz sklep internetowy w którym państwo będą mogli zamawiać towar online, dzięki czemu ułatwi państwu zakupy i przyniesie satysfakcję z udanych zakupów (...)" To tyle co znalazłem na pierwszy rzut oka. Jest tego trochę, ale generalnie sama kosmetyka. Stronka cieszy oko graficznie, choć na tematy grafiki akurat się nie wypowiadam. |
|
|
![]()
Post
#4
|
|
Grupa: Zarejestrowani Postów: 49 Pomógł: 4 Dołączył: 16.07.2008 Skąd: Gdańsk Ostrzeżenie: (0%) ![]() ![]() |
Stronka bardzo fajna ale mam te same uwagi co powyżej plus na stronie głównej aktualności są zbyt ściśnięte - ta grafika po lewej stronie w ogóle tam nie pasuje. W porównaniu z prawą stroną jest ona oderwana od layoutu i dziwne jest że prawa strona znika po przejściu do podstron a lewa zostaje. Ja bym tą grafikę zrobił horyzontalną i wstawił w inne miejsce (może nad menu?). No i proponowałbym zwężenie paska z logo na samej górze o jakieś 15-20%.
|
|
|
![]()
Post
#5
|
|
Grupa: Zarejestrowani Postów: 869 Pomógł: 53 Dołączył: 20.10.2003 Skąd: Przeworsk Ostrzeżenie: (0%) ![]() ![]() |
Skupię się na kodowaniu:
- niepoprawne użycie h1, - puste div-y są fe, jeżeli można jest inne rozwiązanie to warto nad nim popracować, - puste linki? a google to co? jasnowidz?, - puste alt-y: istnieje obrazek, którego nie można opisać?, - niepotrzebne obrazki, które powinny być tłami (np. "Aktualności" itp.), - IMHO bardziej tutaj nadawałby się span, - UTF-8 i > - tak, albo tak. Ogólnie kiepsko. Najbardziej co boli to niepoprawne zastosowanie obrazków jako tekstu. Dla przykładu: Ten kod nic nie mówi google. Czy nie lepiej zamiast obrazka dać tekst, a w stylach schować napis, a obrazek dać jako tło? |
|
|
![]()
Post
#6
|
|
Grupa: Zarejestrowani Postów: 5 Pomógł: 0 Dołączył: 7.03.2009 Ostrzeżenie: (0%) ![]() ![]() |
@Kildyt, dzięki za uwagi. Mam jednak do Ciebie kilka pytań. Dlaczego uważasz, że tag h1 został źle użyty? dlatego, że nie dla głównych kategorii na stronce(np. Aktualności, Opakowania itp.)?
Oczywiście można by było te niepotrzebne obrazki z tekstem zrobić jako tło a dać tekst i margin na -9999, ale czytałem, że nie są to najlepsze praktyki. Jeśli chodzi o pustego diva to jest tylko jeden w topie. Powinno go nie być a grafika mogłaby być w całości ale długo by się to ładowało. Alty oczywiście można dopisać. Niektóre linki prowadzą do nikąd ponieważ nie wszystko zostało skończone. Mówisz, że jest kiepsko. W takim razie jeśli poprawie te błędy to nadal będzie uważał że jest źle? Może ktoś inny się jeszcze wypowie na temat kodowania:) |
|
|
![]()
Post
#7
|
|
Grupa: Zarejestrowani Postów: 869 Pomógł: 53 Dołączył: 20.10.2003 Skąd: Przeworsk Ostrzeżenie: (0%) ![]() ![]() |
@Kildyt, dzięki za uwagi. Mam jednak do Ciebie kilka pytań. Dlaczego uważasz, że tag h1 został źle użyty? dlatego, że nie dla głównych kategorii na stronce(np. Aktualności, Opakowania itp.)? Oczywiście można by było te niepotrzebne obrazki z tekstem zrobić jako tło a dać tekst i margin na -9999, ale czytałem, że nie są to najlepsze praktyki. Jeśli chodzi o pustego diva to jest tylko jeden w topie. Powinno go nie być a grafika mogłaby być w całości ale długo by się to ładowało. Alty oczywiście można dopisać. Niektóre linki prowadzą do nikąd ponieważ nie wszystko zostało skończone. Mówisz, że jest kiepsko. W takim razie jeśli poprawie te błędy to nadal będzie uważał że jest źle? Może ktoś inny się jeszcze wypowie na temat kodowania:) Co do h1 to odsyłam do kursów. Może i text-indent -9999px nie jest najlepszą praktyką, ale na pewno lepszą (dużo) od tego co jest teraz. Jeżeli poprawisz wszystkie błędy to będzie super. (IMG:style_emoticons/default/winksmiley.jpg) |
|
|
![]() ![]() |
![]() |
Aktualny czas: 22.08.2025 - 18:27 |