From 210405f5b6932bbefa5caa02442e3b0218ae1727 Mon Sep 17 00:00:00 2001
From: Aaron Cordovez <aaron.cordovez.m@gmail.com>
Date: Sat, 15 Nov 2014 19:36:34 -0800
Subject: [PATCH] Update labels for best practices

If you nest input tags inside of the label tags you are linking them together. This allows users to click on the label (not only the box) and automatically focus onto the input box. Also, this format makes it easier for Screen Readers to identify forms to fill out on your form.

One other good alternative is to use is to use <label for=""> attribute, e.g.:
<label for="firstname"></label>
<input id="firstname" name="firstname" type="text" value="<%= user ? user.get('firstname') : '' %>">
---
 videos/beginner/index.html | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/videos/beginner/index.html b/videos/beginner/index.html
index 2f1d719e..43937c52 100644
--- a/videos/beginner/index.html
+++ b/videos/beginner/index.html
@@ -44,12 +44,15 @@ <h1>User Manager</h1>
   <script type="text/template" id="edit-user-template">
     <form class="edit-user-form">
       <legend><%= user ? 'Edit' : 'New' %> User</legend>
-        <label>First Name</label>
-        <input name="firstname" type="text" value="<%= user ? user.get('firstname') : '' %>">
-        <label>Last Name</label>
-        <input name="lastname" type="text" value="<%= user ? user.get('lastname') : '' %>">
-        <label>Age</label>
-        <input name="age" type="text" value="<%= user ? user.get('age') : '' %>">
+        <label>First Name
+          <input name="firstname" type="text" value="<%= user ? user.get('firstname') : '' %>">
+        </label>
+        <label>Last Name
+          <input name="lastname" type="text" value="<%= user ? user.get('lastname') : '' %>">
+        </label>
+        <label>Age
+          <input name="age" type="text" value="<%= user ? user.get('age') : '' %>">
+        </label>
         <hr />
        <button type="submit" class="btn"><%= user ? 'Update' : 'Create' %></button>
        <% if(user) { %>