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

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