From 28a8771a37360f7753deb9931c4aa5d85891043a Mon Sep 17 00:00:00 2001
From: Johannes Meyer <johannes.meyer@icinga.com>
Date: Mon, 7 Feb 2022 14:47:03 +0100
Subject: [PATCH 1/2] components/comments.phtml: Use base target `_next` only
 for comment links

---
 .../views/scripts/show/components/comments.phtml            | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/modules/monitoring/application/views/scripts/show/components/comments.phtml b/modules/monitoring/application/views/scripts/show/components/comments.phtml
index 7bc10e507..fd980eeb4 100644
--- a/modules/monitoring/application/views/scripts/show/components/comments.phtml
+++ b/modules/monitoring/application/views/scripts/show/components/comments.phtml
@@ -39,14 +39,14 @@ if (empty($object->comments) && ! $addLink) {
         echo '<br>' . $addLink;
     }
     ?></th>
-    <td data-base-target="_next">
+    <td data-base-target="_self">
     <?php if (empty($object->comments)):
         echo $addLink;
     else: ?>
         <dl class="comment-list">
         <?php foreach ($object->comments as $comment): ?>
             <dt>
-                <a href="<?=  $this->href('monitoring/comment/show', array('comment_id' => $comment->id)) ?>">
+                <a data-base-target="_next" href="<?=  $this->href('monitoring/comment/show', array('comment_id' => $comment->id)) ?>">
                 <?= $this->escape($comment->author) ?>
                 <span class="comment-time">
                     <?= $this->translate('commented') ?>
@@ -59,6 +59,7 @@ if (empty($object->comments) && ! $addLink) {
                     ) ?>
                 <?php endif ?>
                 </span>
+                </a>
                 <?= $comment->persistent ? $this->icon('attach', 'This comment is persistent.') : '' ?>
                 <?php if (isset($delCommentForm)) {
                     // Form is unset if the current user lacks the respective permission
@@ -74,7 +75,6 @@ if (empty($object->comments) && ! $addLink) {
                     );
                     echo $deleteButton;
                 } ?>
-                </a>
             </dt>
             <dd>
                 <?= $this->nl2br($this->createTicketLinks($this->markdownLine($comment->comment, [ 'class' => 'caption']))) ?>

From ae3731f143fe8bd113b0b9be3c82f817ac9ca446 Mon Sep 17 00:00:00 2001
From: Johannes Meyer <johannes.meyer@icinga.com>
Date: Mon, 7 Feb 2022 14:47:45 +0100
Subject: [PATCH 2/2] css: Keep remove-action buttons shown while active

---
 public/css/icinga/main.less | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/public/css/icinga/main.less b/public/css/icinga/main.less
index 763ea1ee9..2173ab782 100644
--- a/public/css/icinga/main.less
+++ b/public/css/icinga/main.less
@@ -155,12 +155,12 @@ a:hover > .icon-cancel {
     &:hover {
       .var(background-color, gray-lightest);
 
-      > .remove-action {
+      > .remove-action button:not(.spinner.active) {
         visibility: visible;
       }
     }
 
-    > .remove-action {
+    > .remove-action button:not(.spinner.active) {
       visibility: hidden;
     }
   }