From 05e1b5b732a664e731b145038ee70ef14aea1acc Mon Sep 17 00:00:00 2001
From: Frank Sweetser <fs@wpi.edu>
Date: Wed, 2 Jun 2010 13:54:59 -0400
Subject: [PATCH] Add icon hints to class add/remove

Icons taken from http://www.freeiconsweb.com/Webmaster-Pixel-Icon-Set.html
---
 app/helpers/application_helper.rb         |    4 ++--
 app/views/puppetclasses/_classes.html.erb |    2 +-
 public/images/add.png                     |  Bin 0 -> 331 bytes
 public/images/application_cascade.png     |  Bin 0 -> 549 bytes
 public/images/delete.png                  |  Bin 0 -> 957 bytes
 5 files changed, 3 insertions(+), 3 deletions(-)
 create mode 100644 public/images/add.png
 create mode 100644 public/images/application_cascade.png
 create mode 100644 public/images/delete.png

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 55b7c72..af670b4 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -36,7 +36,7 @@ module ApplicationHelper
   end
 
   def link_to_remove_puppetclass klass
-    link_to_function klass.name do |page|
+    link_to_function image_tag("delete.png") + " " + klass.name do |page|
       page["selected_puppetclass_#{klass.id}"].remove
       #TODO if the class is already selected, removing it will not add it to the avail class list
       page << "if ($('puppetclass_#{klass.id}')) {"
@@ -47,7 +47,7 @@ module ApplicationHelper
 
   def link_to_add_puppetclass klass, type
     # link to remote is faster than inline js when having many classes
-    link_to_remote klass.klass,
+    link_to_remote "&nbsp;&nbsp;&nbsp;" + image_tag("add.png") + " " + klass.klass,
       :url => assign_puppetclass_path(klass, :type => type),
       :position => {:after => {:success => "selected_classes" }}
   end
diff --git a/app/views/puppetclasses/_classes.html.erb b/app/views/puppetclasses/_classes.html.erb
index 0f012ce..b448d2c 100644
--- a/app/views/puppetclasses/_classes.html.erb
+++ b/app/views/puppetclasses/_classes.html.erb
@@ -2,7 +2,7 @@
   <td id="classes">
     <% group.each do |list| -%>
       <% next if list.nil? %>
-      <li title="Click to expand module class list"><%= link_to_function list.first, toggle_div(list.first) %></li>
+      <li title="Click to expand module class list"><%= link_to_function image_tag("application_cascade.png") + " " + list.first, toggle_div(list.first) %></li>
       <div id=<%= list.first %> style="display: none;">
         <% for klass in list.last.sort -%>
           <% content_tag_for :div, klass, :title => "Click to add #{klass}" do %>
diff --git a/public/images/add.png b/public/images/add.png
new file mode 100644
index 0000000000000000000000000000000000000000..61d25953c2f84175b9787cfa53e2edc08601d8d7
GIT binary patch
literal 331
zcmV-R0kr;!P)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUy?MXyIRCwBA{Qv(y13G9~9W@O|WuggB
zuL)h2g|3!`0Soy1_itvJTMdfXtX(Z21Bo=?&mV?cPxoVpVFOqV`1AV@!>>QTFvN&9
z;P>y}3_pMW#1O-3K*1CjYyrmb^VfG&1AhHt0IEd~@#!qTe*VhL4{yaV`OK{iDE!QT
zdJHuaS5{}TfJ{DoWf=n@fK3fZFUSQ9-+z8*AP0c-LJjyvz5zgIOh0w<0!AR}8mgi2
zFQ31K&5Y@c*pjkej>3O!BV`n^tLN@AY?|_bk+^gK3><9HNSlO2lcWsHIMZ*Oe1MUi
dv8e?JFaU&-kNmLq2fY9Q002ovPDHLkV1lw^iHiUL

literal 0
HcmV?d00001

diff --git a/public/images/application_cascade.png b/public/images/application_cascade.png
new file mode 100644
index 0000000000000000000000000000000000000000..18946e7ddb9f4b42fdf83b3671fa281f435ac93f
GIT binary patch
literal 549
zcmV+=0^0qFP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUz$4Nv%RCwBqQqM~QK@@)b1Ct6i>DHyt
zE`f&-`vbOve?gZ(1HF0Z;IWJ0(PM`Y|A551LJ*z{vbrr;nIUv;cg?`vo!#lp%B;U2
z{opb4X5M`BeKSuel}aZ7kirOKQ1~Br#bWVhdO@}<E0stjB9bJ*GPK)mcF`<>AP9=4
zX|N*J=NDu`h=?`-M?W0L0o%5N+;!ccxLU0)Fv_NB`h_U(px^JuJkOI*6|i8K9^>&i
z)M_;YYbxn<`obsBbsZ?B(Cv0((P%Wx-eUR?pK~zFolYl@(=mZz82$!IU+eXHAc`W)
zE0|Y<7b<}TB=}HOHI~U_gj_D?dq#==$lrkHoTS-oP82cu4CQi}777J;cx%ARXA2x+
zvF&kd&`E<Z5svKu6<rh4ip-g~mB-gx*xxxsD@*%Ad46x@7YKew1JJ(iSM3WWAi&>v
z%4W04y_5~b(;B!Q_5usn!Wdz0VK^8hIDOr?gn*!4uOG4T{7R+7C4mKxVsjyZ?F_*t
zLx0XlBb&SMV-6++9D{Ge?+aN9FV$~i?E-<Ro0B$`)<6^_ki`&$<P9L9tr<IvAoxa(
n<FoBH{&jMd7sRvLp8x{@^^N;Yx`dSt00000NkvXXu0mjf@MHD9

literal 0
HcmV?d00001

diff --git a/public/images/delete.png b/public/images/delete.png
new file mode 100644
index 0000000000000000000000000000000000000000..b622e16c10238f8d16747e9edada5b2e3e64a785
GIT binary patch
literal 957
zcmV;u148_XP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBU#Ur9tkRCwA{lWlBMWfaH%_qJnO+jVQZ
z4eHhs!pOF6VkTrE42X<?AB+kK7-U&gh?)^FCc4c5MuWU`n}Rs7ftW}%`@jdnOCU-_
zM5-nZ3<SG&T}I|cx7ppU-M#H?dtZ95&%I0-Px8yjdG2}s=Q+<khf-A)*`Lqi|8=Wn
zubr582#tgl1Owr(IehE-`xwky)O>-PEqjmVSxg@}o10)=x)_!vZcsFZ<mf}B2ou-+
zkhoOu3;lf^PYGmx`WC;mzSn18Uc1Nh?ha7#F(@npg^TGybddwJwG=2@f}ykL!3^B<
zt^fY3-H!#XEVLdfeZGFbXU8Up_iuuoOkiq|0Uyso`);-uVo_}=N-Ay0e|8<h=WikU
z=iT1s?~Zp+7t4z(^30}4<A<$~L%$&sh+-lb`&Fx#aDz}y%J47SP!C&)tlucFZY^$~
zI0qrYR?;~YZ511uAPoKiW-Nj6!PsT8w{875pPwVPTA^rv&Cb_hU*>|qOJJisvVZ>$
zXVXe(XN@^fFIg;r6z4HHng;QG8_#qz*LJn9p=g7@{;k(w!3<KvVGQ4yh6)OvRYnw*
z8MW9gw5-BIrDKpq8M90Z#J};w+nsB2By;T>+hEE~BRM>X;6N0!@r;+7%|OaZ06V3{
zR?$R7%uF+o1X)wmXjDjMPwQ3`ivNNe3L<oO2JEch-FWhFm#jdP$jNLxuE!GrjPRTQ
znW&mwPAD|(uWQ+Y!t?~VU;smb1Xxz^Zu;_I*VR1-noZ7y3lx$p!wN(I<5~$<6G`YL
zIG?LV-S*8WNJhYh0tknbU^&tI;uoKGDLF;Ea+`Z~11Uj3BB^TrkJM>LNkx8Pex$~2
zfVrwh|4pfp5FSJ%@Vx9L?k96?{ieIIVOOO?MS64?!7&+PMy%5t$>*^3*sR9(#pMQF
zfR_x2<*eYMF{p})64x_uty~U!kpT(+H9TZ;m|^7;FL&>MUl$lZ-HXfnj(lx1Q*ADX
z0j5efidU@0f-)!RV;bW06!>reT<AU~;tFO`itm*p`<}G&n6F#UeCa@UPo9BlFEy(u
zE&%e4j|N60uf=A7Sek+;tH)kF)cL_v!}QN7@8j{j_;HV$m^Q*zLe<>%DB*(E-kU$`
f7@0rL{|PVv+@G1>*Fs_a00000NkvXXu0mjfo|VFK

literal 0
HcmV?d00001

-- 
1.6.6.1