From 9a7edbfa3a79fe34b7e4a4c3ac3b72bd26a5cb14 Mon Sep 17 00:00:00 2001
From: Benjamin White <benjamin.e.white1@gmail.com>
Date: Mon, 11 Jul 2016 16:30:45 -0400
Subject: [PATCH 1/3] it works!

---
 app.js     | 43 +++++++++++++++++++++++++++++++++++++++++++
 index.html |  9 +++++----
 2 files changed, 48 insertions(+), 4 deletions(-)
 create mode 100644 app.js

diff --git a/app.js b/app.js
new file mode 100644
index 00000000..49f4e496
--- /dev/null
+++ b/app.js
@@ -0,0 +1,43 @@
+$(document).ready(function() {
+
+  // logic for adding an item to shopping list
+  $("#js-shopping-list-form").submit(function(event) {
+    // stops default browser behavior for form submission,
+    // since we don't actually want to submit to server
+    event.preventDefault();
+
+    // add new item to botom of list
+    $('.shopping-list').append(
+      '<li>' +
+        '<span class="shopping-item">' + $("#shopping-list-entry").val() + '</span>' +
+        '<div class="shopping-item-controls">' +
+          '<button class="shopping-item-toggle">' +
+            '<span class="button-label">check</span>' +
+          '</button>' +
+          '<button class="shopping-item-delete">' +
+            '<span class="button-label">delete</span>' +
+          '</button>' +
+        '</div>' +
+      '</li>'
+    );
+    // remove the submitted item from the form input
+    $(this)[0].reset();
+  });
+
+  // logic for deleting items from list
+  $(".shopping-list").on("click", ".shopping-item-delete", function(event){
+    // here `this` refers to the `.shopping-item-delete` element that was clicked.
+    // we travel up the document tree to get the nearest parent element
+    // that"s an `li`
+    $(this).closest("li").remove();
+  });
+
+  // logic for checking/unchecking items
+  $(".shopping-list").on("click", ".shopping-item-toggle", function(event){
+
+    // toggle the .shopping-item__checked class
+    $(this).closest("li").find(".shopping-item").toggleClass(
+      "shopping-item__checked");
+  });
+
+})
\ No newline at end of file
diff --git a/index.html b/index.html
index 17eed56c..cbbb5e7d 100644
--- a/index.html
+++ b/index.html
@@ -13,9 +13,9 @@
   <div class="container">
     <h1>Shopping List</h1>
 
-    <form>
-      <label for="shopping-list-item">Add an item</label>
-      <input type="text" name="shopping-list-item" id="shopping-list-item" placeholder="e.g., broccoli">
+    <form id="js-shopping-list-form">
+      <label for="shopping-list-entry">Add an item</label>
+      <input type="text" name="shopping-list-entry" id="shopping-list-entry" placeholder="e.g., broccoli">
       <button type="submit">Add item</button>
     </form>
 
@@ -66,6 +66,7 @@ <h1>Shopping List</h1>
       </li>
     </ul>
   </div>
-
+  <script src="https://code.jquery.com/jquery-3.1.0.min.js"></script>
+  <script type="text/javascript" src="app.js"></script>
 </body>
 </html>
\ No newline at end of file

From 51b44ac1d1f05825928ca919bc5ea8f77dd04b3f Mon Sep 17 00:00:00 2001
From: Benjamin White <benjamin.e.white1@gmail.com>
Date: Mon, 11 Jul 2016 18:40:37 -0400
Subject: [PATCH 2/3] update text

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index cbbb5e7d..a6f0e276 100644
--- a/index.html
+++ b/index.html
@@ -46,7 +46,7 @@ <h1>Shopping List</h1>
         <span class="shopping-item shopping-item__checked">milk</span>
         <div class="shopping-item-controls">
           <button class="shopping-item-toggle">
-            <span class="button-label">uncheck</span>
+            <span class="button-label">check</span>
           </button>
           <button class="shopping-item-delete">
             <span class="button-label">delete</span>

From 9a5045ed14002b867932fd702dcb5edab6c2d1bc Mon Sep 17 00:00:00 2001
From: Aric Johnson <aric87@comcast.net>
Date: Mon, 11 Jul 2016 23:46:46 -0400
Subject: [PATCH 3/3] convert quotes, remove extra params

using 2 different quotes is sloppy (and I yell at the students all the time for it). Switched to single.
removed the extra event parameters. better not to create the extra room in memory for the unused vars
fixed typo in bottom
put toggleClass onto one line (idk, looks prettier)
---
 app.js | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/app.js b/app.js
index 49f4e496..5a2a868f 100644
--- a/app.js
+++ b/app.js
@@ -1,12 +1,12 @@
 $(document).ready(function() {
 
   // logic for adding an item to shopping list
-  $("#js-shopping-list-form").submit(function(event) {
+  $('#js-shopping-list-form').submit(function(event) {
     // stops default browser behavior for form submission,
     // since we don't actually want to submit to server
     event.preventDefault();
 
-    // add new item to botom of list
+    // add new item to bottom of list
     $('.shopping-list').append(
       '<li>' +
         '<span class="shopping-item">' + $("#shopping-list-entry").val() + '</span>' +
@@ -21,23 +21,22 @@ $(document).ready(function() {
       '</li>'
     );
     // remove the submitted item from the form input
-    $(this)[0].reset();
+    $(this)[0].reset(); 
   });
 
   // logic for deleting items from list
-  $(".shopping-list").on("click", ".shopping-item-delete", function(event){
+  $('.shopping-list').on('click', '.shopping-item-delete', function(){
     // here `this` refers to the `.shopping-item-delete` element that was clicked.
     // we travel up the document tree to get the nearest parent element
     // that"s an `li`
-    $(this).closest("li").remove();
+    $(this).closest('li').remove();
   });
 
   // logic for checking/unchecking items
-  $(".shopping-list").on("click", ".shopping-item-toggle", function(event){
+  $('.shopping-list').on('click', '.shopping-item-toggle', function(){
 
     // toggle the .shopping-item__checked class
-    $(this).closest("li").find(".shopping-item").toggleClass(
-      "shopping-item__checked");
+    $(this).closest('li').find('.shopping-item').toggleClass('shopping-item__checked');
   });
 
-})
\ No newline at end of file
+})