[Rt-commit] rt branch, 5.0-trunk, updated. rt-5.0.0-78-gb013a4be3b

? sunnavy sunnavy at bestpractical.com
Wed Nov 4 14:24:22 EST 2020


The branch, 5.0-trunk has been updated
       via  b013a4be3b5a57bd5cbf916ee27c4e1614f86673 (commit)
       via  a47d2ec230408ce72ddf68c9e61db558619e8f53 (commit)
       via  707d75f1213c30928ff2726b1ee720697beb1d77 (commit)
       via  4eecf190d9236c40b14ab53bca87a1b57e72db6f (commit)
       via  920357ce361f3b8fc44b8d5a833b81dbebe6c8ea (commit)
      from  79a837a35aaad4e86cedfcee28284915fb48b7c8 (commit)

Summary of changes:
 share/html/Elements/BulkCustomFields | 12 +++++-------
 share/html/Search/Bulk.html          | 30 ++++++++++++------------------
 2 files changed, 17 insertions(+), 25 deletions(-)

- Log -----------------------------------------------------------------
commit 920357ce361f3b8fc44b8d5a833b81dbebe6c8ea
Author: Blaine Motsinger <blaine at bestpractical.com>
Date:   Tue Nov 3 16:55:47 2020 -0600

    Center comment/reply messagebox on bulk update
    
    This commit centers the messagebox on the bulk update page to match
    the layout of the Ticket/Update page.

diff --git a/share/html/Search/Bulk.html b/share/html/Search/Bulk.html
index 491eb12e1d..335678a1eb 100644
--- a/share/html/Search/Bulk.html
+++ b/share/html/Search/Bulk.html
@@ -290,7 +290,7 @@
 </&>
 <&| /Widgets/TitleBox, title => loc('Add comments or replies to selected tickets') &>
   <div class="form-row">
-    <div class="label col-3">
+    <div class="form-group label col-2">
       <&|/l&>Update Type</&>:
     </div>
     <div class="value col-9">
@@ -302,7 +302,7 @@
   </div>
 
   <div class="form-row">
-    <div class="label col-3">
+    <div class="form-group label col-2">
       <&|/l&>Subject</&>:
     </div>
     <div class="value col-9">
@@ -314,7 +314,7 @@
 % $m->callback( CallbackName => 'BeforeTransactionCustomFields', CustomFields => $TxnCFs );
 % while (my $CF = $TxnCFs->Next()) {
   <div class="form-row">
-    <div class="label col-3">
+    <div class="label col-2">
       <% $CF->Name %>:
     </div>
     <div class="value col-9">
@@ -326,11 +326,8 @@
   </div>
 % } # end if while
 
-  <div class="form-row">
-    <div class="label col-3">
-      <&|/l&>Message</&>:
-    </div>
-    <div class="value col-9 messagebox-container action-<% $ARGS{UpdateType} || 'private' %>">
+  <div class="form-group">
+    <div class="messagebox-container action-<% $ARGS{UpdateType} || 'private' %>">
 %# Currently, bulk update always starts with Comment not Reply selected, so we check this unconditionally
 % my $IncludeSignature = RT->Config->Get('MessageBoxIncludeSignatureOnComment');
       <& /Elements/MessageBox, Name => "UpdateContent", 
@@ -341,13 +338,7 @@
   </div>
 % $m->callback( %ARGS, CallbackName => 'BeforeMessageBox' );
 
-  <div class="form-row">
-    <div class="label col-3"></div>
-    <div class="col-9">
-      <& /Ticket/Elements/AddAttachments, %ARGS &>
-    </div>
-  </div>
-
+  <& /Ticket/Elements/AddAttachments, %ARGS &>
 </&>
 
 <%perl>

commit 4eecf190d9236c40b14ab53bca87a1b57e72db6f
Author: Blaine Motsinger <blaine at bestpractical.com>
Date:   Tue Nov 3 17:20:17 2020 -0600

    Replace cf hints with tooltips on bulk update
    
    This commit replaces the custom field hints with tooltips on the
    bulk update page.

diff --git a/share/html/Elements/BulkCustomFields b/share/html/Elements/BulkCustomFields
index 02beed3ff1..21ade01c55 100644
--- a/share/html/Elements/BulkCustomFields
+++ b/share/html/Elements/BulkCustomFields
@@ -62,12 +62,10 @@
 % while (my $cf = $CustomFields->Next) {
   <div class="form-row <% ++$i%2 ? 'oddline': 'evenline' %>">
     <div class="label col-4 text-left">
-      <div class="form-row">
-        <% $cf->Name %>
-      </div>
-      <div class="form-row">
-        <em>(<% $cf->EntryHint // '' %>)</em>
-      </div>
+      <span class="name"><% $cf->Name %>:</span>
+% if ( $cf->EntryHint ) {
+      <span class="far fa-question-circle icon-helper" data-toggle="tooltip" data-placement="top" data-original-title="<% $cf->EntryHint %>"></span>
+% }
     </div>
 % my $rows = 5;
 % my $cf_id = $cf->id;
diff --git a/share/html/Search/Bulk.html b/share/html/Search/Bulk.html
index 335678a1eb..5b37f57803 100644
--- a/share/html/Search/Bulk.html
+++ b/share/html/Search/Bulk.html
@@ -315,13 +315,16 @@
 % while (my $CF = $TxnCFs->Next()) {
   <div class="form-row">
     <div class="label col-2">
-      <% $CF->Name %>:
+      <span class="name"><% $CF->Name %>:</span>
+% if ( $CF->EntryHint ) {
+      <span class="far fa-question-circle icon-helper" data-toggle="tooltip" data-placement="top" data-original-title="<% $CF->EntryHint %>"></span>
+% }
     </div>
     <div class="value col-9">
       <& /Elements/EditCustomField,
         CustomField => $CF,
         Object => RT::Transaction->new( $session{'CurrentUser'} ),
-      &><em><% $CF->EntryHint // '' %></em>
+      &>
     </div>
   </div>
 % } # end if while

commit 707d75f1213c30928ff2726b1ee720697beb1d77
Author: Blaine Motsinger <blaine at bestpractical.com>
Date:   Tue Nov 3 17:25:14 2020 -0600

    Fix BeforeMessageBox callback location
    
    The BeforeMessageBox callback on the bulk update page was
    incorrectly moved below the messagebox in commit 547407b.
    
    This commit moves it back to it's correct location.

diff --git a/share/html/Search/Bulk.html b/share/html/Search/Bulk.html
index 5b37f57803..17251c6ad4 100644
--- a/share/html/Search/Bulk.html
+++ b/share/html/Search/Bulk.html
@@ -331,6 +331,7 @@
 
   <div class="form-group">
     <div class="messagebox-container action-<% $ARGS{UpdateType} || 'private' %>">
+% $m->callback( %ARGS, CallbackName => 'BeforeMessageBox' );
 %# Currently, bulk update always starts with Comment not Reply selected, so we check this unconditionally
 % my $IncludeSignature = RT->Config->Get('MessageBoxIncludeSignatureOnComment');
       <& /Elements/MessageBox, Name => "UpdateContent", 
@@ -339,7 +340,6 @@
       &>
     </div>
   </div>
-% $m->callback( %ARGS, CallbackName => 'BeforeMessageBox' );
 
   <& /Ticket/Elements/AddAttachments, %ARGS &>
 </&>

commit a47d2ec230408ce72ddf68c9e61db558619e8f53
Author: Blaine Motsinger <blaine at bestpractical.com>
Date:   Tue Nov 3 18:21:09 2020 -0600

    Fix padding for custom field inputs on bulk update

diff --git a/share/html/Elements/BulkCustomFields b/share/html/Elements/BulkCustomFields
index 21ade01c55..83e4218615 100644
--- a/share/html/Elements/BulkCustomFields
+++ b/share/html/Elements/BulkCustomFields
@@ -60,7 +60,7 @@
   </div>
 % my $i = 0;
 % while (my $cf = $CustomFields->Next) {
-  <div class="form-row <% ++$i%2 ? 'oddline': 'evenline' %>">
+  <div class="form-row py-2 <% ++$i%2 ? 'oddline': 'evenline' %>">
     <div class="label col-4 text-left">
       <span class="name"><% $cf->Name %>:</span>
 % if ( $cf->EntryHint ) {
diff --git a/share/html/Search/Bulk.html b/share/html/Search/Bulk.html
index 17251c6ad4..aae57a0e51 100644
--- a/share/html/Search/Bulk.html
+++ b/share/html/Search/Bulk.html
@@ -314,7 +314,7 @@
 % $m->callback( CallbackName => 'BeforeTransactionCustomFields', CustomFields => $TxnCFs );
 % while (my $CF = $TxnCFs->Next()) {
   <div class="form-row">
-    <div class="label col-2">
+    <div class="form-group label col-2">
       <span class="name"><% $CF->Name %>:</span>
 % if ( $CF->EntryHint ) {
       <span class="far fa-question-circle icon-helper" data-toggle="tooltip" data-placement="top" data-original-title="<% $CF->EntryHint %>"></span>

commit b013a4be3b5a57bd5cbf916ee27c4e1614f86673
Merge: 79a837a35a a47d2ec230
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Thu Nov 5 03:15:40 2020 +0800

    Merge branch '5.0/center-comment-reply-on-bulk-update-page' into 5.0-trunk


-----------------------------------------------------------------------


More information about the rt-commit mailing list