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!

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.

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

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.

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 :)!

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

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.

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/

Jo det funkar nog med Modernizr! Så det kanske är ett bättre alternativ :)

Jag får hoppas att du menar tidigare och inte senare. :D

Funkar inte i Internet Explorer 9 heller.. Kanske 10an, har själv inte testat.

Men placeholder är väl jquery baserat?... Det ska ju fungera oavsett version och tillverkare?

Antar att dom menar <input placeholder="Detta är en placeholder">

Inget direkt fel på typsnittet men ta bort all text-shadow, förstår inte på något plan varför du har det där.

( text-shadow:0 0.0625em #5F5F5F )

Tänk även på att kolla hur bra hinting det är på typsnittet du använder då webkit-läsare kan rendera vissa typsnitt suddigt, använd helst en tjänst som typekit eller google fonts så slipper du att tänka på att uppdatera typsnittet för hinting.

Försök optimera din css en del. Saker som vilket typsnitt som skall användas ska inte behöva finnas i "header nav ul li p". Hade inte heller använt id's i samma utsträckning
utan försökt hålla mig till klasser så dom inte skriver över dina klasser, bara en smaksak men märkt att det blir mycket effektivare kod på detta sätt.

Haha, givetvis :D

Allvarligt? Jag har t.o.m fått denna att snurra felfritt på IE9
http://icase.emildayan.se/

Jag tycker du skall ha kvar hover-effekten på loggan.

Jag håller med om att du borde ta bort skuggan på texten. Blir mer svårläst med den

Förstår inte vad det har med saken att göra. Det är ju olika saker. När det gäller <input placeholder="blabla"> så nej, har själv inte fått det att funka i IE9. :(

Konstigt att en sån liten grej som placeholder ej fungerar i IE9 när jag lyckats fått ett helt Canvas (html5) projekt att fungera felfritt - det var min poäng. Dessutom tror jag att placeholder fungerar hur bra som helst på IE, se bara till att lägga till http-equiv metan för att tvinga IE ur compatibility-mode.

Nej jag provade men det funkade inte.. Håller med om att det är konstigt och jag förstår din poäng.

Tack för era synpunkter, vi får se hur det blir med fonten jag försökte uppnå photoshops "crisp" funktioner genom att ändra på bl.a. skuggan.

Jag tycker verkligen du ska ta bort skuggan på brödtexten eller åtminstone kolla över den lite mer. Just nu ger det mig en feeling som inte alls är bra. Jag orkar inte läsa texten på din sida i alla fall. Den ser verkligen inte bra ut, den blir otroligt otydlig. Måste vara skuggan som gör det hela.

Vill inte vara så negativ för att sidan i övrigt var väldigt fin och stilren, men just brödtexten bör du nog se över.

Annars som sagt bra jobbat, fin och stilren!