Pod adresem: http://henmar.com.pl/sklep/ znajduje się sklep internetowy producenta kołder.
Prosiłbym o konstruktywne opinie, sugestie co zmienić, poprawić, usprawnić.
Pełna wersja: [sklep] Henmar
Usunąć:
Taka drobnostka: dobry by był anty-aliasing dla obrazku na góra-prawo
Inna drobnostka: "Asortyment" - mogłaby być jakaś inna czcionka bo dla mnie wygląda jakbyś zapomniał ją sformatować
Wygląd generalnie nie jest zły. W kod się nie zagłębiałem, ale powierzchownie też nie jest zły. Waliduje się.
A i brakuje mi stopki...
Cytat
Fatal error: Call to a member function FetchRow() on a non-object in /usr/local/apache/www/htdocs/henmar.com.pl/sklep/lib/nowosciLista.lib.php on line 40
Taka drobnostka: dobry by był anty-aliasing dla obrazku na góra-prawo
Inna drobnostka: "Asortyment" - mogłaby być jakaś inna czcionka bo dla mnie wygląda jakbyś zapomniał ją sformatować
Wygląd generalnie nie jest zły. W kod się nie zagłębiałem, ale powierzchownie też nie jest zły. Waliduje się.
A i brakuje mi stopki...
flash do menu to przesada .
zbyt wielki margin w stopce
blok - logowanie/rejestracja | koszyk psuje stronę , strasznie pusty , taki wygląd jest ok jak robisz stronę dla siebie i masz gdzieś wygląd bo ma tylko coś działać . Tu jest inaczej.
zrób coś z ' Asortyment ' nie pasuje do reszty
zbyt wielki margin w stopce
blok - logowanie/rejestracja | koszyk psuje stronę , strasznie pusty , taki wygląd jest ok jak robisz stronę dla siebie i masz gdzieś wygląd bo ma tylko coś działać . Tu jest inaczej.
zrób coś z ' Asortyment ' nie pasuje do reszty
Pyro, gdzie wylookałeś poniższy monit? Na stronie głównej, czy pod linkiem nowosci.php?
Kod
Fatal error: Call to a member function FetchRow() on a non-object in /usr/local/apache/www/htdocs/henmar.com.pl/sklep/lib/nowosciLista.lib.php on line 40
Cytat(Savage.Mephisto @ 21.05.2009, 20:41:59 ) 
Pyro, gdzie wylookałeś poniższy monit? Na stronie głównej, czy pod linkiem nowosci.php?
Kod
Fatal error: Call to a member function FetchRow() on a non-object in /usr/local/apache/www/htdocs/henmar.com.pl/sklep/lib/nowosciLista.lib.php on line 40
http://henmar.com.pl/sklep/nowosci.php?start=4'
Śmiem podejrzewać, że istnieje tu podatność SQL Injection
// edit
Użyj intval()" title="Zobacz w manualu PHP" target="_manual
Dzięki za wnikliwe badanie kodu
Niedługo zrobię update.
Niedługo zrobię update.
Te stopy w topie nie wyglądają zbyt apetycznie. Do tego można Cię posądzić o szerzenie pornografii dziecęcej - stopy dziecka i mężczyzny są na jednym poziomie rodzi się więc pytanie: gdzie jest głowa dziecka?
PS: te poduszeczki na dole jakoś tak nie pasują.
PS: te poduszeczki na dole jakoś tak nie pasują.
Cytat(phpion @ 22.05.2009, 09:38:00 )
Te stopy w topie nie wyglądają zbyt apetycznie. Do tego można Cię posądzić o szerzenie pornografii dziecęcej - stopy dziecka i mężczyzny są na jednym poziomie rodzi się więc pytanie: gdzie jest głowa dziecka?
Jeśli wg Ciebie wygląda to na szerzenie pornografii dziecięcej, to z całym szacunkiem gratuluję wybujałej wyobraźni.
mi osobiście nie pasuje :
Te divy z samym paddingiem są zbędne...
style lepiej umieszczać w css
Niepotrzebny Div dla samego jednego wyrazu...
rozmiar, czcionka - Identyczne
Bardziej zagłębiać mi się nie chce - ogólnie fajnie moja ocena (3+).
Ale to tylko moje zdanie...
Kod
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
<dt class="hidden"></dt>
Kod
<div style="padding-top: 10px;"></div>
<div class="container_newprod"></div><div class="container_prod">
<div style="font-size: 9px; border: solid 2px #d7d7d7; width: 106px; height: 80px;"><a href="produkt.php?prodId=22"><img src="galeria/thumb/thumb_1235604054.jpg" width="106" height="80" title="thumb_1235604054.jpg" alt="thumb_1235604054.jpg" /></a></div></div>
<div style="padding-top: 10px;"></div>
<span class="link1"><strong>Cena: </strong>308.00 zł</span>
<div class="container_newprod"></div><div class="container_prod">
<div style="font-size: 9px; border: solid 2px #d7d7d7; width: 106px; height: 80px;"><a href="produkt.php?prodId=22"><img src="galeria/thumb/thumb_1235604054.jpg" width="106" height="80" title="thumb_1235604054.jpg" alt="thumb_1235604054.jpg" /></a></div></div>
<div style="padding-top: 10px;"></div>
<span class="link1"><strong>Cena: </strong>308.00 zł</span>
Te divy z samym paddingiem są zbędne...
Kod
<span style="font-weight: bold; font-size: 14px;">1</span>
style lepiej umieszczać w css
Kod
<div class="naglowek">
<h3>Nowości</h3>
</div>
<h3>Nowości</h3>
</div>
Niepotrzebny Div dla samego jednego wyrazu...
Kod
<p style="font-size: 14px; font-family: Arial; text-align: justify;">Zakład Przetwórstwa Pierza Henmar s.c. mieści się w Wierzbicy koło Radomia. Zakład powstał w 1995 r. Podstawową działalnością zakładu jest produkcja wyrobów pościelowych. Produkcja obejmuje kołdry, poduszki, jaśki na bazie puchu i pierza gęsiego oraz kołdry zdrowotne. Zakład wykonuje dodatkowo usługi w zakresie renowacji używanej pościeli.</p>
<p style="font-size: 14px; font-family: Arial; text-align: right;"> Zapraszamy do zakupów !</p
<p style="font-size: 14px; font-family: Arial; text-align: right;"> Zapraszamy do zakupów !</p
rozmiar, czcionka - Identyczne
Bardziej zagłębiać mi się nie chce - ogólnie fajnie moja ocena (3+).
Ale to tylko moje zdanie...
Cytat(Poker @ 22.05.2009, 12:45:27 )
Kod
<p style="font-size: 14px; font-family: Arial; text-align: justify;">Zakład Przetwórstwa Pierza Henmar s.c. mieści się w Wierzbicy koło Radomia. Zakład powstał w 1995 r. Podstawową działalnością zakładu jest produkcja wyrobów pościelowych. Produkcja obejmuje kołdry, poduszki, jaśki na bazie puchu i pierza gęsiego oraz kołdry zdrowotne. Zakład wykonuje dodatkowo usługi w zakresie renowacji używanej pościeli.</p>
<p style="font-size: 14px; font-family: Arial; text-align: right;"> Zapraszamy do zakupów !</p
<p style="font-size: 14px; font-family: Arial; text-align: right;"> Zapraszamy do zakupów !</p
rozmiar, czcionka - Identyczne
Bardziej zagłębiać mi się nie chce - ogólnie fajnie moja ocena (3+).
Ale to tylko moje zdanie...
Poker, dzięki za tak wnikliwe zagłębienie się w kod. Dziś wieczorem siądę i poprawię go nieco.
Odnośnie tego samego rozmiaru i rodzaju czcionki w tekście Zakład Przetwórstwa Pierza..., to jest to wynikiem użycia edytora TinyMCE z poziomu panelu adm. Czasem tworzy on 'zupę' z tagów.
Design nie zachwyca. Jest średni. Menu górne jest IMHO ulokowane w złym miejscu. Teraz wydaje się one mniej ważne od reszty, a powinno być najważniejsze. Logotyp ulokowałbym raczej z lewej strony, jak jest to na większości stron. Dodałbym również stopkę z zastrzeżonymi prawami autorskimi. Komunikat błędu (np. o obowiązku rejestracji przed wystawieniem recenzji) jest zbyt rażąca.
W kodzie html jest zbyt dużo stylów.
Tabela w miejscu koszyka chyba nie jest najlepszym pomysłem.
Używasz h3, gdy nie ma h1, ani h2.
Rozgranicz znacznik b i strong.
W kodzie html jest zbyt dużo stylów.
nie lepiej wrzucić klasę do linka?
Tabela w miejscu koszyka chyba nie jest najlepszym pomysłem.
Używasz h3, gdy nie ma h1, ani h2.
Rozgranicz znacznik b i strong.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.