From b6a09c9ebb448584bb469206bf1aaab7638c6d4e Mon Sep 17 00:00:00 2001
From: cuom1999 <lephuocdinh99@gmail.com>
Date: Sat, 19 Nov 2022 17:30:07 -0600
Subject: [PATCH] Update actionbar share and comment

---
 resources/libs/clipboard/tooltip.js        |  4 ++--
 templates/actionbar/list.html              |  8 ++++----
 templates/actionbar/media-js.html          | 14 ++++++++++++++
 templates/base.html                        |  1 +
 templates/blog/content.html                |  1 +
 templates/comments/list.html               | 11 ++++++++---
 templates/comments/media-js.html           | 17 ++---------------
 templates/common-content.html              |  1 -
 templates/problem/feed.html                |  1 +
 templates/submission/status-testcases.html |  4 ++--
 10 files changed, 35 insertions(+), 27 deletions(-)

diff --git a/resources/libs/clipboard/tooltip.js b/resources/libs/clipboard/tooltip.js
index 6333457..9cce976 100644
--- a/resources/libs/clipboard/tooltip.js
+++ b/resources/libs/clipboard/tooltip.js
@@ -1,5 +1,5 @@
-function showTooltip(elem, msg) {
-    elem.setAttribute('class', 'btn-clipboard tooltipped tooltipped-s');
+function showTooltip(elem, msg, direction='s') {
+    $(elem).addClass('tooltipped tooltipped-' + direction);
     elem.setAttribute('aria-label', msg);
 }
 function fallbackMessage(action) {
diff --git a/templates/actionbar/list.html b/templates/actionbar/list.html
index 64b8389..d3ca294 100644
--- a/templates/actionbar/list.html
+++ b/templates/actionbar/list.html
@@ -17,8 +17,8 @@
         </span>
     </span>
     {% if not hide_actionbar_comment %}
-    <span class="actionbar-block" id="comment-button">
-        <span class="actionbar-button">
+    <span class="actionbar-block">
+        <span class="actionbar-button actionbar-comment">
             <i class="fa fa-comment-o" style="font-size: large;"></i>
             <span class="actionbar-text">{{_("Comment")}}</span>
         </span>
@@ -33,8 +33,8 @@
             <span class="actionbar-text">{{_("Bookmark")}}</span>
         </span>
     </span>
-    <span class="actionbar-block" id="share-button" >
-        <span class="actionbar-button">
+    <span class="actionbar-block" >
+        <span class="actionbar-button actionbar-share" style="position: relative" {{"share-url=" + share_url if share_url else ""}}>
             <i class="fa fa-share" style="font-size: large;"></i>
             <span class="actionbar-text">{{_("Share")}}</span>
         </span>
diff --git a/templates/actionbar/media-js.html b/templates/actionbar/media-js.html
index 9144ea1..1e3b083 100644
--- a/templates/actionbar/media-js.html
+++ b/templates/actionbar/media-js.html
@@ -104,7 +104,21 @@
                     }
                 }
             };
+            $(".actionbar-share").click( function() {
+                link = $(this).attr("share-url") || window.location.href;
+                navigator.clipboard
+                         .writeText(link)
+                         .then(() =>  {
+                            showTooltip(this, "Copied link", 'n');
+                        });
+            });
 
+            $('.actionbar-comment').on('click', function() {
+                if ($('#comment-announcement').length) {
+                    $('#comment-announcement').click();
+                }
+                $('#write-comment').click();
+            })
         });
     </script>
 {% endcompress %}
\ No newline at end of file
diff --git a/templates/base.html b/templates/base.html
index b771565..a9b3d6b 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -91,6 +91,7 @@
         <script src="{{ static('libs/select2/select2.js') }}"></script>
         {% include "extra_js.html" %}
         <script src="{{ static('common.js') }}"></script>
+        <script src="{{ static('libs/clipboard/tooltip.js') }}"></script>
         <script>
             moment.locale('{{ LANGUAGE_CODE }}');
             $(function () {
diff --git a/templates/blog/content.html b/templates/blog/content.html
index ba754f0..506271e 100644
--- a/templates/blog/content.html
+++ b/templates/blog/content.html
@@ -46,6 +46,7 @@
         {% set bookmark = post.bookmark %}
         {% set hide_actionbar_comment = True %}
         {% set include_hr = True %}
+        {% set share_url = request.build_absolute_uri(post.get_absolute_url()) %}
         {% include "actionbar/list.html" %}
     </div>
 </section>
\ No newline at end of file
diff --git a/templates/comments/list.html b/templates/comments/list.html
index db61a76..7cafe3e 100644
--- a/templates/comments/list.html
+++ b/templates/comments/list.html
@@ -1,7 +1,12 @@
+{% set can_comment = request.user.is_authenticated and comment_form and not comment_lock %}
 <div id="comments" class="comment-area">
-    <h2 id="comment-header"><i style="padding-right: 0.3em" class="fa fa-comments"></i>{{ _('Comments') }}</h2>
-    <p> <a href="" id="write-comments" > Write comments </a> </p>
-    {% if request.user.is_authenticated and comment_form and not comment_lock %}
+    <h2 id="comment-header">
+        <i style="padding-right: 0.3em" class="fa fa-comments"></i>{{ _('Comments') }}
+        {% if can_comment %}
+            <a href="" id="write-comment" style="float: right; font-size: 0.6em; margin-right: -26px;"> {{ _('Write comment') }} </a>
+        {% endif %}
+    </h2>
+    {% if can_comment %}
         <div id="new-comment" class="form-area comment-submit" style="display: none;" >
             {% block comment_submit_title %}
                 <h3>{{ _('New comment') }}</h3>
diff --git a/templates/comments/media-js.html b/templates/comments/media-js.html
index 4e92461..0fbfeae 100644
--- a/templates/comments/media-js.html
+++ b/templates/comments/media-js.html
@@ -212,25 +212,12 @@
                 $comment.find('.bad-comment-body').hide();
             };
             
-            $("#share-button").click( function() {
-                navigator.clipboard
-                         .writeText(window.location.href)
-                         .then( () =>  {
-                            alert("Copied link to this post");
-                        });
-            });
-
-            $("#write-comments").click( function(event)  {
+            $("#write-comment").click( function(event)  {
                 event.preventDefault();
                 $("#new-comment").show("slow");
-                $("#write-comments").hide();
+                $("#write-comment").hide();
             });
             
-            $('#comment-button').click( function() {
-                $('#comment-section').show();
-                $('#comment-announcement').hide();    
-            })
-             
         });
     </script>   
 {% endcompress %}
diff --git a/templates/common-content.html b/templates/common-content.html
index c809252..52f195b 100644
--- a/templates/common-content.html
+++ b/templates/common-content.html
@@ -7,7 +7,6 @@
 {% block js_media %}
     {% compress js %}
         <script src="{{ static('libs/clipboard/clipboard.js') }}"></script>
-        <script src="{{ static('libs/clipboard/tooltip.js') }}"></script>
         <script type="text/javascript">
             $(function () {
                 var info_float = $('.info-float');
diff --git a/templates/problem/feed.html b/templates/problem/feed.html
index 888ae88..848e6c9 100644
--- a/templates/problem/feed.html
+++ b/templates/problem/feed.html
@@ -65,6 +65,7 @@
                 {% set hide_actionbar_comment = True %}
                 {% set pagevote = problem.pagevote %}
                 {% set bookmark = problem.bookmark %}
+                {% set share_url = request.build_absolute_uri(problem.get_absolute_url()) %}
                 {% include "actionbar/list.html" %}
 
                 {% if feed_type=='volunteer' and request.user.has_perm('judge.suggest_problem_changes') %}
diff --git a/templates/submission/status-testcases.html b/templates/submission/status-testcases.html
index 0af451a..f4d7b15 100644
--- a/templates/submission/status-testcases.html
+++ b/templates/submission/status-testcases.html
@@ -28,7 +28,7 @@
         {% for batch in batches %}
             <div style="margin-top: 0.5em;">
                 {% if batch.id %}
-                    <b>{{ _('Batch ') }}#{{ loop.index }}: </b>
+                    <b>{{ _('Batch ') }}#{{ batch.id }}: </b>
                 {% endif %}
                 {% for case in batch.cases %}
                     <span class="case-{{ case.status }} case-icons">
@@ -86,7 +86,7 @@
                 {% set batch_AC = (batch.points == batch.total) %}
                 <tr class="case-row" style="background-color: {{ '#d6e8f7' if batch.points == batch.total else 'bisque'}}">
                     <td>
-                        <b>{{ _('Batch ') }}#{{ loop.index }}</b>
+                        <b>{{ _('Batch ') }}#{{ batch.id }}</b>
                     </td>
                     <td>
                         {% if batch_AC %}