From ba0ee581c467b95a350413df21847a7b4d7570e5 Mon Sep 17 00:00:00 2001
From: Andreas Nedbal <git@pixelde.su>
Date: Tue, 30 Aug 2022 19:47:18 +0200
Subject: [PATCH] Remove "frown" state when an answer/comment is smiled

---
 .../features/answerbox/comment/smile.ts          |  2 --
 .../retrospring/features/answerbox/smile.ts      |  2 --
 app/views/answerbox/_actions.haml                | 16 +++-------------
 app/views/answerbox/_comments.haml               | 16 +++-------------
 4 files changed, 6 insertions(+), 30 deletions(-)

diff --git a/app/javascript/retrospring/features/answerbox/comment/smile.ts b/app/javascript/retrospring/features/answerbox/comment/smile.ts
index c0ed442a..f78c8747 100644
--- a/app/javascript/retrospring/features/answerbox/comment/smile.ts
+++ b/app/javascript/retrospring/features/answerbox/comment/smile.ts
@@ -47,11 +47,9 @@ export function commentSmileHandler(event: Event): void {
         switch(action) {
           case 'smile':
             button.dataset.action = 'unsmile';
-            button.innerHTML = `<i class="fa fa-fw fa-frown-o"></i> <span id="ab-comment-smile-count-${id}">${count}</span>`;
             break;
           case 'unsmile':
             button.dataset.action = 'smile';
-            button.innerHTML = `<i class="fa fa-fw fa-smile-o"></i> <span id="ab-comment-smile-count-${id}">${count}</span>`;
             break;
         }
       }
diff --git a/app/javascript/retrospring/features/answerbox/smile.ts b/app/javascript/retrospring/features/answerbox/smile.ts
index de99db7e..9f49144f 100644
--- a/app/javascript/retrospring/features/answerbox/smile.ts
+++ b/app/javascript/retrospring/features/answerbox/smile.ts
@@ -47,11 +47,9 @@ export function answerboxSmileHandler(event: Event): void {
         switch(action) {
           case 'smile':
             button.dataset.action = 'unsmile';
-            button.innerHTML = `<i class="fa fa-fw fa-frown-o"></i> <span id="ab-smile-count-${id}">${count}</span>`;
             break;
           case 'unsmile':
             button.dataset.action = 'smile';
-            button.innerHTML = `<i class="fa fa-fw fa-smile-o"></i> <span id="ab-smile-count-${id}">${count}</span>`;
             break;
         }
       }
diff --git a/app/views/answerbox/_actions.haml b/app/views/answerbox/_actions.haml
index 6d61b837..6f2050b9 100644
--- a/app/views/answerbox/_actions.haml
+++ b/app/views/answerbox/_actions.haml
@@ -1,17 +1,7 @@
 %span.d-none.d-sm-inline.text-muted
-  - if !user_signed_in? && a.smiles.count.positive?
-    %button.btn.btn-info.btn-sm{ name: "ab-smile", disabled: true }
-      %i.fa.fa-smile-o
-      %span{ id: "ab-smile-count-#{a.id}" }= a.smiles.count
-- if user_signed_in?
-  - if current_user.smiled? a
-    %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-smile", data: { a_id: a.id, action: :unsmile } }
-      %i.fa.fa-fw.fa-frown-o
-      %span{ id: "ab-smile-count-#{a.id}" }= a.smiles.count
-  - else
-    %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-smile", data: { a_id: a.id, action: :smile } }
-      %i.fa.fa-fw.fa-smile-o
-      %span{ id: "ab-smile-count-#{a.id}" }= a.smiles.count
+  %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-smile", data: { a_id: a.id, action: current_user&.smiled?(a) ? :unsmile : :smile }, disabled: !user_signed_in? }
+    %i.fa.fa-fw.fa-smile-o
+    %span{ id: "ab-smile-count-#{a.id}" }= a.smiles.count
 - unless display_all
   %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-comments", data: { a_id: a.id, state: :hidden } }
     %i.fa.fa-fw.fa-comments
diff --git a/app/views/answerbox/_comments.haml b/app/views/answerbox/_comments.haml
index 0c87c165..7193c7a5 100644
--- a/app/views/answerbox/_comments.haml
+++ b/app/views/answerbox/_comments.haml
@@ -18,19 +18,9 @@
               = markdown comment.content
           .pull-right
             %span.d-none.d-sm-inline.text-muted
-              - if !user_signed_in? && comment.smile_count.positive?
-                %button.btn.btn-link.answerbox__action{ name: "ab-smile-comment", disabled: true }
-                  %i.fa.fa-fw.fa-smile-o
-                  %span{ id: "ab-comment-smile-count-#{comment.id}" }= comment.smile_count
-            - if user_signed_in?
-              - if current_user.smiled? comment
-                %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-smile-comment", data: { c_id: comment.id, action: :unsmile } }
-                  %i.fa.fa-fw.fa-frown-o
-                  %span{ id: "ab-comment-smile-count-#{comment.id}" }= comment.smile_count
-              - else
-                %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-smile-comment", data: { c_id: comment.id, action: :smile } }
-                  %i.fa.fa-fw.fa-smile-o
-                  %span{ id: "ab-comment-smile-count-#{comment.id}" }= comment.smile_count
+              %button.btn.btn-link.answerbox__action{ type: :button, name: "ab-smile-comment", data: { c_id: comment.id, action: current_user&.smiled?(comment) ? :unsmile : :smile }, disabled: !user_signed_in? }
+                %i.fa.fa-fw.fa-smile-o
+                %span{ id: "ab-comment-smile-count-#{comment.id}" }= comment.smile_count
               .btn-group
                 %button.btn.btn-link.btn-sm.dropdown-toggle{ data: { toggle: :dropdown }, aria: { expanded: false } }
                   %span.caret