From 1b7dedf9d7029db81378962b9524177d2f232d0b Mon Sep 17 00:00:00 2001
From: Jared Hancock <jared@osticket.com>
Date: Mon, 16 Dec 2013 14:43:34 -0600
Subject: [PATCH] Fix UI bug where user change warning always showed
HTML does not allow nested <p> tags. Therefore the user warning was not
correctly hidden with the rest of the user change confirmation message.
---
include/staff/ticket-view.inc.php | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/include/staff/ticket-view.inc.php b/include/staff/ticket-view.inc.php
index 3cee43a88..0cf0e0dff 100644
--- a/include/staff/ticket-view.inc.php
+++ b/include/staff/ticket-view.inc.php
@@ -897,9 +897,10 @@ $tcount+= $ticket->getNumNotes();
Are you sure want to <b>unassign</b> ticket from <b><?php echo $ticket->getAssigned(); ?></b>?
</p>
<p class="confirm-action" style="display:none;" id="changeuser-confirm">
- <p id="msg_warning">
- <b><?php echo Format::htmlchars($ticket->getName()); ?></b> <<?php echo $ticket->getEmail(); ?>> will no longer have access to the ticket.
- </p>
+ <span id="msg_warning" style="display:block;vertical-align:top">
+ <b><?php echo Format::htmlchars($ticket->getName()); ?></b> <<?php echo $ticket->getEmail(); ?>>
+ <br> will no longer have access to the ticket.
+ </span>
Are you sure want to <b>change</b> ticket owner to <b><span id="newuser">this guy</span></b>?
</p>
<p class="confirm-action" style="display:none;" id="delete-confirm">
--
GitLab