WN

WN (https://www.wn.se/forum/index.php)
-   Feedback (https://www.wn.se/forum/forumdisplay.php?f=20)
-   -   Griffin Studio (https://www.wn.se/forum/showthread.php?t=1057327)

MorganB 2013-03-24 17:27

Griffin Studio
 
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. :rolleyes:

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 2013-03-24 18:22

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 2013-03-24 18:46

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 2013-03-24 19:00

Citat:

Ursprungligen postat av yakuzaemme (Inlägg 20465678)
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 2013-03-24 19:43

Citat:

Ursprungligen postat av MorganB (Inlägg 20465686)
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 2013-03-24 19:49

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 2013-03-24 20:13

Citat:

Ursprungligen postat av Westman (Inlägg 20465691)
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 2013-03-24 20:21

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 2013-03-24 20:24

Citat:

Ursprungligen postat av yakuzaemme (Inlägg 20465690)
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 2013-03-24 20:33

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/


Alla tider är GMT +2. Klockan är nu 11:40.

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