From d7214dc9417545c8a58504c6fc2fa05bd3e38a1d Mon Sep 17 00:00:00 2001
From: Milka Kuzminski <milka@rhodecode.com>
Date: Fri, 9 Apr 2021 11:27:31 +0200
Subject: [PATCH] draft: fixed logic in toggle all draft for submit.

---
 rhodecode/templates/pullrequests/pullrequest_show.mako | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/rhodecode/templates/pullrequests/pullrequest_show.mako b/rhodecode/templates/pullrequests/pullrequest_show.mako
index 125faf4e..1ed75a78 100644
--- a/rhodecode/templates/pullrequests/pullrequest_show.mako
+++ b/rhodecode/templates/pullrequests/pullrequest_show.mako
@@ -560,7 +560,7 @@
 
                 <div class="right-sidebar-expanded-state pr-details-title">
                   <span style="padding-left: 2px">
-                  <input name="select_all_drafts" type="checkbox" onclick="$('[name=submit_draft]').prop('checked', !$('[name=submit_draft]').prop('checked'))">
+                  <input name="select_all_drafts" type="checkbox" onclick="selectDraftComments(event)">
                   </span>
                   <span class="sidebar-heading noselect" onclick="refreshDraftComments(); return false">
                     <i class="icon-comment icon-draft"></i>
@@ -942,6 +942,11 @@ window.setObserversData = ${c.pull_request_set_observers_data_json | n};
         }
     }
 
+    window.selectDraftComments = function (event) {
+        var $target = $(event.currentTarget);
+        $('[name=submit_draft]').prop('checked', $target.prop('checked'))
+    }
+
     window.closePullRequest = function (status) {
         if (!confirm(_gettext('Are you sure to close this pull request without merging?'))) {
             return false;