[sklep] Henmar, sugestie, opinie mile widziane Opcje

[Savage.Mephisto]

Pod adresem: http://henmar.com.pl/sklep/ znajduje się sklep internetowy producenta kołder.
Prosiłbym o konstruktywne opinie, sugestie co zmienić, poprawić, usprawnić.

[pyro]

Usunąć:

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

Ten post edytował pyro 21.05.2009, 19:01:52

[Spawnm]

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

[Savage.Mephisto]

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

Ten post edytował Savage.Mephisto 21.05.2009, 19:42:57

[pyro]

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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

// edit

Użyj intval()" title="Zobacz w manualu PHP" target="_manual

Ten post edytował pyro 21.05.2009, 21:18:08

[Savage.Mephisto]

Dzięki za wnikliwe badanie kodu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Niedługo zrobię update.

[phpion]

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

[Savage.Mephisto]

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.

[Poker]

mi osobiście nie pasuje :

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>

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>

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>

Niepotrzebny Div dla samego jednego wyrazu...

Kod

<p style="font-size: 14px; font-family: Arial; text-align: justify;">Zakład Przetw&oacute;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&oacute;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;">&nbsp;Zapraszamy do zakup&oacute;w !</p

rozmiar, czcionka - Identyczne





Bardziej zagłębiać mi się nie chce - ogólnie fajnie moja ocena (3+).


Ale to tylko moje zdanie...

[Savage.Mephisto]

Kod

<p style="font-size: 14px; font-family: Arial; text-align: justify;">Zakład Przetw&oacute;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&oacute;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;">&nbsp;Zapraszamy do zakup&oacute;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.

Ten post edytował Savage.Mephisto 22.05.2009, 14:44:03

[Kildyt]

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.

[HTML] pobierz, plaintext 
<span class="link2"><a href="login.php">
[HTML] pobierz, plaintext 

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.