[MorganB]

Hej!

Jag har skrapat ihop en portfolio på http://griffinstudio.se skulle uppskatta allt möjlig feedback jag kan få av er. Både positiv och negativ så jag vet vad jag kan förbättra eller vad jag redan har gjort på ett bra sätt som ni tycker är ett plus.

Jag är mest för webbdesign/front-end och behärskar en hel del men just att designa hemsidor tycker jag är super kul. Jag skulle uppskatta om ni har några förslag på hur man kan komma in i marknaden, jag vill som sagt kunna skapa hemsidor som även andra kan ha nytta för och inte bara jag. Ser helst även inte att priset spelar någon större roll, då jag utöver allt detta i form av en hobbyverksamhet.

Ha en bra dag!

[yakuzaemme]

Snyggt!

Felstavning på sista meningen på startsidan. Tycker även du kan ta bort hover på logan.

Skulle även skriva om näst intill all JS-kod (http://griffinstudio.se/js/funktion.js) då uppbyggnaden tyvärr inte ser bra ut alls.

[simonwellander]

Jag tyckte det också såg bra ut men håller med yakuzaemme om hover-effekten på loggan.

Gillar illustrationerna och grafiken på sidan! Och även "min typiska vardag".

[MorganB]

Ursprungligen postat av yakuzaemme

Snyggt!

Felstavning på sista meningen på startsidan. Tycker även du kan ta bort hover på logan.

Skulle även skriva om näst intill all JS-kod (http://griffinstudio.se/js/funktion.js) då uppbyggnaden tyvärr inte ser bra ut alls.

Tack så mycket! Det är fixat nu. Jag ska nog tänka över logan. JS är inte min starkaste sida men ska jobba lite mer på det så släpper nog snart en förbättring på koden !

[yakuzaemme]

Ursprungligen postat av MorganB

Tack så mycket! Det är fixat nu. Jag ska nog tänka över logan. JS är inte min starkaste sida men ska jobba lite mer på det så släpper nog snart en förbättring på koden !

Jag menar mer logiken på HTML samt JS-koden.

Exempelvis din första funktion, send-message. Använd placeholder istället för att göra värdet på input till "Namn" eller "Meddelande".

Du har 3 error-divs på kontaktformuläret. Gör en klass istället som har ikonen, och om en input har error så lägger du bara till klassen 'error' på det elementet.

Skulle även ändra url från '/ommig' till '/om'

Sidan är annars väldigt "rolig" i sig, såsom min vardag m.m. Kanske du ska styla lite mer, möjligtvis ladda din e-postadress med AJAX så crawlers ej kan hitta den, sen kan du även skriva ut något såsom "Take that crawlers!" - nej men du förstår. Visa lite kunskaper på den fronten också, även fast det lutar lite mer åt backend.

[Westman]

Vi tar en liten svenskläraret först: "föddes över havs" är felformulerat. Antingen till havs (på ett fartyg t.ex.) eller över havet (på ett flygplan).

Jag har också svårt för fonten, får en stark känsla av obehag pga. att jag upplever den oskarp och jobbig att läsa.

Småsaker med andra ord.

[MorganB]

Ursprungligen postat av Westman

Vi tar en liten svenskläraret först: "föddes över havs" är felformulerat. Antingen till havs (på ett fartyg t.ex.) eller över havet (på ett flygplan).

Jag har också svårt för fonten, får en stark känsla av obehag pga. att jag upplever den oskarp och jobbig att läsa.

Småsaker med andra ord.

Haha nämen tack! Vad skulle jag gjort utan dig, det är rättat nu !

[simonwellander]

Nu är jag inte så bra på javascript (jobbar på det), men det känns som du använder onödigt mycket javascript-kod på ställen där du skulle klarat dig utan? Precis som i yakuzaemme exempel (men just HTML5 placeholder funkar ju inte i alla webbläsare).

[MorganB]

Ursprungligen postat av yakuzaemme

Jag menar mer logiken på HTML samt JS-koden.

Exempelvis din första funktion, send-message. Använd placeholder istället för att göra värdet på input till "Namn" eller "Meddelande".

Du har 3 error-divs på kontaktformuläret. Gör en klass istället som har ikonen, och om en input har error så lägger du bara till klassen 'error' på det elementet.

Skulle även ändra url från '/ommig' till '/om'

Sidan är annars väldigt "rolig" i sig, såsom min vardag m.m. Kanske du ska styla lite mer, möjligtvis ladda din e-postadress med AJAX så crawlers ej kan hitta den, sen kan du även skriva ut något såsom "Take that crawlers!" - nej men du förstår. Visa lite kunskaper på den fronten också, även fast det lutar lite mer åt backend.

Jag ska titta på det! Btw är inte placeholder bara med i HTML5? Jag hade med old school js scriptet med där bara för den fungerar i princip på alla gamla webbläsare med.

[yakuzaemme]

Det är väl bara IE 8 och senare som ej stödjer HTML5? Finns ju mängder med fallbacks, såsom http://modernizr.com/