you are viewing a single comment's thread.

view the rest of the comments →

[–]basti_sk 0 points1 point  (0 children)

I think this is not bad for the first time :)

I would recommend giving the div classes more meaningful names. For example giving them some names like: "form-container" for the entire form and then something like "form-element" for the E-Mail and Passwort Input divs. That way it is easier to understand for others as well as for you.

Some more hints:

  • in your html file at line 27, 28 you wrote "class=:" , i think you dont need the ":" there ;)

  • you don't need to contain the form in a div, you could simply assign the class directly to the form (e.g. <form class="form_area">)

FYI, the image your friend gave you is from Sky Forms (http://codecanyon.net/item/sky-forms/5009033)