Merge pull request #1551 from NARKOZ/front-end

add auto-completion for +1, -1 emoji
2012-09-25 10:12:32 -07:00 · 2012-09-25 10:12:32 -07:00 · 2cc8d91dad
parent c3efcf7a62 6df515a56d
commit 2cc8d91dad
1 changed files with 8 additions and 2 deletions
--- a/app/views/notes/_common_form.html.haml
+++ b/app/views/notes/_common_form.html.haml
@ -39,6 +39,12 @@

 :javascript
  $(function(){
-    var names = #{@project.users.pluck(:name)};
-    $('.note-text').atWho('@', { data: names });
+    var names = #{@project.users.pluck(:name)}, emoji = ['+1', '-1'];
+    var emoji = $.map(emoji, function(value, i) {return {key:value + ':', name:value}});
+    $('#note_note').
+      atWho('@', { data: names }).
+      atWho(':', {
+        data: emoji,
+        tpl: "<li data-value='${key}'>${name} #{escape_javascript image_tag('emoji/${name}.png', :size => '20x20')}</li>"
+      });
  });