From 59a5c60f3b714d874e6c62c15da39c6bb1e4a4b9 Mon Sep 17 00:00:00 2001
From: Matchu <matchu1993@gmail.com>
Date: Mon, 7 Jan 2013 18:55:36 -0600
Subject: [PATCH] i18n for outfits/edit.js item partials: no-data-yet and icons

---
 app/views/outfits/edit.html.haml   | 20 +++++++++++++
 config/locales/en-meep.yml         |  1 +
 config/locales/en.yml              |  1 +
 public/javascripts/outfits/edit.js | 45 ++++++++----------------------
 4 files changed, 33 insertions(+), 34 deletions(-)

diff --git a/app/views/outfits/edit.html.haml b/app/views/outfits/edit.html.haml
index 7a832842..acbfaad1 100644
--- a/app/views/outfits/edit.html.haml
+++ b/app/views/outfits/edit.html.haml
@@ -151,6 +151,26 @@
       %a.outfit-delete-confirmation-no{:href => '#'}
         = t '.sidebar.outfits.outfit.delete_confirmation.cancel'
     </li>
+  %script#item-template{:type => 'text/x-jquery-tmpl'}
+    %li{:class => 'object object-${id}'}
+      %img{:src => '${thumbnail_url}', :title => '${description}', :alt => ''}
+      ${name}
+      %ul
+      = link_to '${url}', :target => '_blank', :class => 'object-info' do
+        %span i
+      {{if nc}}
+      = nc_icon
+      {{/if}}
+      {{if owned || wanted}}
+      .closeted-icons
+        {{if owned}}
+        = owned_icon
+        {{/if}}
+        {{if wanted}}
+        = wanted_icon
+        {{/if}}
+      {{/if}}
+      %span.no-assets-message= t '.sidebar.closet.no_data.summary'
 
 %span#userbar-session-message
   - if user_signed_in?
diff --git a/config/locales/en-meep.yml b/config/locales/en-meep.yml
index b1203972..a98d516a 100644
--- a/config/locales/en-meep.yml
+++ b/config/locales/en-meep.yml
@@ -486,6 +486,7 @@ en-meep:
               success: Outfeet successfully meeped
         closet:
           no_data:
+            summary: No meepits yet
             description:
               We haven't meeped this item on this body type before. Have you?
               Meep it for us on the home page if you have!
diff --git a/config/locales/en.yml b/config/locales/en.yml
index b3e69ca8..3204d6fe 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -487,6 +487,7 @@ en:
               success: Outfit successfully saved
         closet:
           no_data:
+            summary: No data yet
             description:
               We haven't seen this item on this body type before. Have you?
               Model it for us on the home page if you have!
diff --git a/public/javascripts/outfits/edit.js b/public/javascripts/outfits/edit.js
index 50d73c4f..58bde75d 100644
--- a/public/javascripts/outfits/edit.js
+++ b/public/javascripts/outfits/edit.js
@@ -26,7 +26,7 @@ var Partial = {}, main_wardrobe,
 Partial.ItemSet = function ItemSet(wardrobe, selector) {
   var item_set = this, ul = $(selector), items = [], setClosetItems,
     setOutfitItems, setOutfitItemsControls, no_assets_full_message = $('#no-assets-full-message'),
-    container = $('#container');
+    container = $('#container'), item_template = $('#item-template');
 
   Partial.ItemSet.setWardrobe(wardrobe);
 
@@ -59,13 +59,6 @@ Partial.ItemSet = function ItemSet(wardrobe, selector) {
       item = specific_items[i];
       no_assets = item.couldNotLoadAssetsFitting(wardrobe.outfits.getPetType());
       li = $('li.object-' + item.id).toggleClass('no-assets', no_assets);
-      (function (li) {
-        no_assets_message = li.find('span.no-assets-message');
-        no_assets_message.remove();
-        if(no_assets) {
-          $('<span/>', {'class': 'no-assets-message', text: 'No data yet'}).appendTo(li);
-        }
-      })(li);
     }
   }
 
@@ -75,33 +68,17 @@ Partial.ItemSet = function ItemSet(wardrobe, selector) {
     ul.children().remove();
     for(var i = 0; i < items.length; i++) {
       item = items[i];
-      li = $('<li/>', {'class': 'object object-' + item.id});
-      img = $('<img/>', {
-        'src': item.thumbnail_url,
-        'alt': item.description,
-        'title': item.description
+      li = item_template.tmpl({
+        id: item.id,
+        name: item.name,
+        thumbnail_url: item.thumbnail_url,
+        url: item.getURL(),
+        nc: typeof item.rarity_index != 'undefined' &&
+            (item.rarity_index == 500 || item.rarity_index == 0),
+        owned: item.owned,
+        wanted: item.wanted
       });
-      controls = $('<ul/>');
-      info_link = $('<a/>', {
-        'class': 'object-info',
-        html: '<span>i</span>',
-        href: item.getURL(),
-        target: '_blank'
-      });
-      if(
-        typeof item.rarity_index != 'undefined' &&
-        (item.rarity_index == 500 || item.rarity_index == 0)
-      ) {
-        $('<div/>', {'class': 'nc-icon', text: 'NC', title: 'NC'}).appendTo(li);
-      }
-      if(item.owned || item.wanted) {
-        var iconsWrapper = $('<div/>', {'class': 'closeted-icons'}).appendTo(li);
-        if(item.owned) {
-          $('<img/>', {alt: 'Own', title: 'You own this', src: '/images/owned.png'}).appendTo(iconsWrapper);
-          $('<img/>', {alt: 'Want', title: 'You want this', src: '/images/wanted.png'}).appendTo(iconsWrapper);
-        }
-      }
-      li.append(img).append(controls).append(info_link).append(item.name).appendTo(ul);
+      li.appendTo(ul);
     }
     setClosetItems(wardrobe.outfits.getClosetItems());
     setOutfitItems(wardrobe.outfits.getWornItems());