From 0aca529e277617dba305677ce436d444fd1beeda Mon Sep 17 00:00:00 2001
From: Matchu <matchusian@gmail.com>
Date: Sat, 26 Sep 2015 13:56:25 -0700
Subject: [PATCH] checkbox for closet management

drop the remove link (just use quantity: 0)
to create a bigger checkbox label target
---
 app/assets/javascripts/closet_hangers/index.js | 11 ++++++++++-
 .../stylesheets/closet_hangers/_index.sass     | 18 ++++++++++++++++++
 app/views/closet_hangers/index.html.haml       |  3 +--
 3 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/app/assets/javascripts/closet_hangers/index.js b/app/assets/javascripts/closet_hangers/index.js
index cb5c114b..78630540 100644
--- a/app/assets/javascripts/closet_hangers/index.js
+++ b/app/assets/javascripts/closet_hangers/index.js
@@ -91,7 +91,16 @@
           var hangerId = hangerEl.data("id");
           var quantityEl = hangerEl.find("div.quantity");
           var quantity = hangerEl.data("quantity");
-          
+
+          // Ooh, this part is weird. We only want the name to be linked, so
+          // lift everything else out.
+          var label = $('<label />');
+          var checkbox = $('<input type="checkbox" />').appendTo(label);
+          var link = hangerEl.children('a');
+          link.children(':not(.name)').detach().appendTo(label);
+          link.detach().appendTo(label);
+          label.appendTo(hangerEl);
+
           $.tmpl("updateFormTmpl", {
             user_id: currentUserId,
             closet_hanger_id: hangerId,
diff --git a/app/assets/stylesheets/closet_hangers/_index.sass b/app/assets/stylesheets/closet_hangers/_index.sass
index d6c9afb2..76dd1c3e 100644
--- a/app/assets/stylesheets/closet_hangers/_index.sass
+++ b/app/assets/stylesheets/closet_hangers/_index.sass
@@ -112,6 +112,21 @@ body.closet_hangers-index
         .quantity
           display: none
 
+      a
+        /* It's jarring to have line-height gaps in the linkiness. */
+        display: block
+
+        &:hover
+          text-decoration: underline
+
+      input[type=checkbox]
+        display: none
+        position: absolute
+        top: 0
+        right: ($object-width - $object-img-size) / 2 + $object-padding
+        height: 16px
+        width: 16px
+
   .closet-hangers-group
     border-top: 1px solid $module-border-color
     margin-bottom: 2em
@@ -315,6 +330,9 @@ body.closet_hangers-index
           input[type=submit]
             font-size: 85%
 
+        input[type=checkbox]
+          display: block
+
     &.js
       #closet-hangers
         .object:hover .quantity
diff --git a/app/views/closet_hangers/index.html.haml b/app/views/closet_hangers/index.html.haml
index 19f1532e..4aa5ae89 100644
--- a/app/views/closet_hangers/index.html.haml
+++ b/app/views/closet_hangers/index.html.haml
@@ -121,8 +121,7 @@
                            min: 0, required: true
 
     %script#closet-hanger-destroy-tmpl{type: 'text/x-jquery-tmpl'}
-      = form_tag templated_hanger_path, method: :delete, authenticity_token: false, class: 'closet-hanger-destroy' do
-        = submit_tag t('closet_hangers.closet_hanger.delete')
+      -# TODO: remove me?
 
 - content_for :stylesheets do
   = stylesheet_link_tag 'http://ajax.googleapis.com/ajax/libs/jqueryui/1.9.0/themes/south-street/jquery-ui.css'