From b6c0baf44d7d737ede7784848374a8af3e911ac1 Mon Sep 17 00:00:00 2001
From: Dan Brown <ssddanbrown@googlemail.com>
Date: Mon, 7 Oct 2019 20:21:04 +0100
Subject: [PATCH] Updated comment delete action to be a button

Fixes issue that causes code error when an anchor tag.

Closes #1650
---
 resources/assets/js/components/page-comments.js | 2 ++
 resources/views/comments/comment.blade.php      | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/resources/assets/js/components/page-comments.js b/resources/assets/js/components/page-comments.js
index cabce9139..5d8d16958 100644
--- a/resources/assets/js/components/page-comments.js
+++ b/resources/assets/js/components/page-comments.js
@@ -26,10 +26,12 @@ class PageComments {
 
     handleAction(event) {
         let actionElem = event.target.closest('[action]');
+
         if (event.target.matches('a[href^="#"]')) {
             const id = event.target.href.split('#')[1];
             scrollAndHighlightElement(document.querySelector('#' + id));
         }
+
         if (actionElem === null) return;
         event.preventDefault();
 
diff --git a/resources/views/comments/comment.blade.php b/resources/views/comments/comment.blade.php
index 5fbdfa500..ea96a9250 100644
--- a/resources/views/comments/comment.blade.php
+++ b/resources/views/comments/comment.blade.php
@@ -31,7 +31,7 @@
                         <button type="button" dropdown-toggle aria-haspopup="true" aria-expanded="false" class="text-button" title="{{ trans('common.delete') }}">@icon('delete')</button>
                         <ul class="dropdown-menu" role="menu">
                             <li class="px-m text-small text-muted pb-s">{{trans('entities.comment_delete_confirm')}}</li>
-                            <li><a action="delete" href="#" class="text-button text-neg" >@icon('delete'){{ trans('common.delete') }}</a></li>
+                            <li><button action="delete" type="button" class="text-button text-neg" >@icon('delete'){{ trans('common.delete') }}</button></li>
                         </ul>
                     </div>
                 @endif