WN

WN (https://www.wn.se/forum/index.php)
-   Feedback (https://www.wn.se/forum/forumdisplay.php?f=20)
-   -   feedback på min egen utvecklad e-handel (https://www.wn.se/forum/showthread.php?t=1044411)

nabil_akhlaque 2010-10-21 16:56

feedback på min egen utvecklad e-handel
 
Hejsan,

äntligen börjar jag bli klar med min e-handelslösning.
skulle vilja har kreativ kritik på allt.. dvs utformning, design, upplägget.

ps. jag vet att den är lite seg... ;)

ni kan logga in med
anv jj
lösen: 12

url:
vvvpunktminmatbutikpunktcom

Fredrik-s 2010-10-21 17:10

Ser riktigt bra ut. Förstår att du har lagt ner ett väldigt jobb för att producera detta, bra jobbat!

najk 2010-10-21 17:16

Snyggt jobbat Nabil, säkert ett hästjobb bakom!
Funkar inte så bra i Safari att lägga saker i kundvagnen dock. Även om effekten va lagom pedagogisk. Rättare sagt, ingenting verkar funkar i Safari, försöker klicka på den där fyrkantslayoutgrejen, snurrar men händer inget.

rhdf 2010-10-21 18:41

säkert ett monsterjobb bakom detta, men tyvärr så genereras monstruöst mycket onödigt "skräp" i koden pga att du byggt allt med de inbyggda kontrollerna i asp.net

Att du sedan väljer att bygga layouten med tabeller gör det inte bättre

Cosma 2010-10-22 07:30

Jättesnyggt jobbat. Dock skulle jag önska en cursor: pointer på dina gröna knappar ;)

Softilog 2010-10-22 08:46

Ser bra ut.
Skulle dock vilja ha möjlighet att:
- söka på tex leverantör.
- filtrera/sortera resultatlistorna på tex leverantör, namn, pris etc.
- Välja hur många varor som skall visas i resultatlistorna

Jag kan inte se vad en Matkasse innehåller (firefox)

allstars 2010-10-22 10:27

Layoutmässigt hade jag önskat lite mer luft i menyn och produktboxar.

Markupmässigt inte använda tabeller och istället göra css-replace för priset. Det är inte tillgängligt som det är nu.

Allt hängde sig när jag gick in på en produkt.
Varför har du använt en textarea för produktbeskrivningen, eller vad används den till?
Utseendet på priset på produktsidan borde följa som det ser ut i produktboxar.

plura 2010-10-22 11:05

Riktigt jävla snygg! Fick genast känsla av ICA-butik!

Liten ändring bara:
Priserna på förstasidan ser väldigt utsmetade ut.

Kaugesaar 2010-10-22 11:11

Ser bra ut, upplägget, design etc etc.

Men om vi skall gå in på tekniska detaljer är det en ren mardröm SEO-mässigt.

Undoredoo 2010-10-22 14:25

Första intrycket var verkligen "wow". Ambitiöst, seriöst, stiligt.

Men jag kan inte låta bli att uppröras av att se så dålig kod år 2010. Ingen tutorial eller mänskliga lärare har ju lärt ut tabellbaserad layout på bra många år, så jag förstår inte att det fortfarande dyker upp. I synnerhet när det gäller nyproduktion. Jag kan dessvärre inte vara speciellt konstruktiv på den punkten. Förlåt.

Sen är det som sagt långa svarstider. Först tänkte jag att varukategorierna till vänster expanderas via AJAX-anrop, och att det alltså var servern som var orsaken. Men det verkar ju inte vara något annat än display-toggle via javascript. Hur kan det ta så lång tid?

Men den absolut viktigaste frågan är: har du någonting med www.minmatbutik.se att göra? Svaret på den frågan står och faller med din trovärdighet, enligt mig.

Eller också är det jag som är helt fel ute och/eller bara är på dåligt humör. Ber om ursäkt i så fall.

rhdf 2010-10-22 14:45

Undoredoo: anledningen till svarstiderna i kategoriträdet är detta
<a href="javascript:__doPostBack('TreeView1','tBlommo r')">
det görs med andra ord en full postback varje gång man klickar på nånting i den
att det inte märks på sidan beror på att trädmenyn ligger i en "updatepanel"
nackdelen med den lilla "uppfinningen" från .net-teamet är att ALL kod på den aktuella sidan körs likförbannat på servern.

nabil_akhlaque 2010-10-22 16:22

tjenare..
yes, det var min gamla asp version som ligger kvar på .se, men som ska ersättas med .net versionen när den blir klar ;)


Citat:

Ursprungligen postat av Undoredoo (Inlägg 20374726)
Första intrycket var verkligen "wow". Ambitiöst, seriöst, stiligt.

Men jag kan inte låta bli att uppröras av att se så dålig kod år 2010. Ingen tutorial eller mänskliga lärare har ju lärt ut tabellbaserad layout på bra många år, så jag förstår inte att det fortfarande dyker upp. I synnerhet när det gäller nyproduktion. Jag kan dessvärre inte vara speciellt konstruktiv på den punkten. Förlåt.

Sen är det som sagt långa svarstider. Först tänkte jag att varukategorierna till vänster expanderas via AJAX-anrop, och att det alltså var servern som var orsaken. Men det verkar ju inte vara något annat än display-toggle via javascript. Hur kan det ta så lång tid?

Men den absolut viktigaste frågan är: har du någonting med www.minmatbutik.se att göra? Svaret på den frågan står och faller med din trovärdighet, enligt mig.

Eller också är det jag som är helt fel ute och/eller bara är på dåligt humör. Ber om ursäkt i så fall.


Undoredoo 2010-10-22 17:24

Citat:

Ursprungligen postat av nabil_akhlaque (Inlägg 20374750)
tjenare..
yes, det var min gamla asp version som ligger kvar på .se, men som ska ersättas med .net versionen när den blir klar ;)

Ok, jag är nöjd. :)

Men du borde fortfarande göra något åt gränssnittskoden. På riktigt.

FlipDesign 2010-11-03 16:54

hmm, visst ser den bra ut och så men sidan är öppen för sql injections.
Testa att logga in med ' or '1'='1 som användarnamn och lösenord då kommer man in på administratörskontot.

Iggy 2010-11-07 21:13

Designen var utan tvekan snygg, men uppbyggnaden är en av de värsta jag sett på en hemsida som till och med är gjord det här året.
  • Dissa tabeller över huvud taget.
  • Lägg gärna all text med teckensnittet "ICA" i bilder så att det är säkert att alla ser lika.
  • Säkra sidan från SQL-injektioner (!) - var inget problem alls att komma in på administratörkontot. Att ha ett inloggningssystem utan säkerhet är som att köra bil utan bromsar.
  • Måste verkligen allt vara uppbyggt med JavaScript? Sidan skulle tjäna på att istället köra med validerad HTML, både ur SEO- och användarsynpunkt.

Sen så finns det en hel del småsaker som skulle kunna optimeras - ett exempel är att lägga alla ikoner (inklusive hover) i en och samma bild och sedan visa dem som bakgrunder, och att du kör med UTF-8 istället för ISO-8859-1.

På något sätt irriterar jag mig lite på den där laddningssymbolen, men det kanske bara är jag.

rhdf 2010-11-08 03:11

lite intressant att du använt de inbyggda kontrollerna i .net till precis ALLT, oavsett om det behövs eller inte, utom just inloggningen. Den inbyggda membership-providern har ju skydd mot SQL-injections.

jayzee 2010-11-08 07:43

Fixa till din UpdatePanel4 i Default.aspx. Jag får System.Exception: No database specified.


Alla tider är GMT +2. Klockan är nu 12:38.

Programvara från: vBulletin® Version 3.8.2
Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
Svensk översättning av: Anders Pettersson