[Rt-commit] rt branch, 4.2/onetime-checkbox-wrapping, created. rt-4.2.13-55-g840fbe6

Dustin Collins strega at bestpractical.com
Wed Sep 7 07:39:45 EDT 2016


The branch, 4.2/onetime-checkbox-wrapping has been created
        at  840fbe67b5ac6e28d95af119cb4ea4fc4ecf6651 (commit)

- Log -----------------------------------------------------------------
commit 840fbe67b5ac6e28d95af119cb4ea4fc4ecf6651
Author: Dustin Collins <strega at bestpractical.com>
Date:   Wed Sep 7 07:33:47 2016 -0400

    Prevent Cc checkbox and label from wrapping independently
    
    One-Time Cc and Bcc checkboxes wrap independently of their associated label making it difficult to determine which input belongs to which label. In order to maintain a more clear visual association, both input and label elements are wrapped in a nowrap styled span.
    
    Because this association should be maintained across themes, the style is defined inline as an html attribute.
    
    Fixes: I#32117

diff --git a/share/html/Ticket/Elements/UpdateCc b/share/html/Ticket/Elements/UpdateCc
index 6101139..7e36e19 100644
--- a/share/html/Ticket/Elements/UpdateCc
+++ b/share/html/Ticket/Elements/UpdateCc
@@ -55,6 +55,7 @@
 <i class="label">(<&|/l&>check to add</&>)</i>
 %}
 %foreach my $addr ( @one_time_Ccs ) {
+<span style="white-space: nowrap;">
 <input
     id="UpdateCc-<%$addr%>"
     name="UpdateCc-<%$addr%>"
@@ -64,6 +65,7 @@
     onClick="checkboxToInput('UpdateCc', <% "UpdateCc-$addr" |n,j%>, <%$clean_addr|n,j%> );"
     <% $ARGS{'UpdateCc-'.$addr} ? 'checked="checked"' : ''%> >
       <label for="UpdateCc-<%$addr%>"><& /Elements/ShowUser, Address => $txn_addresses{$addr}&></label>
+</span>
 %}
 </td></tr>
 <tr><td class="label"><&|/l&>One-time Bcc</&>:</td><td><& /Elements/EmailInput, Name => 'UpdateBcc', Size => undef, Default => $ARGS{UpdateBcc}, AutocompleteMultiple => 1 &><br />
@@ -71,6 +73,7 @@
 <i class="label">(<&|/l&>check to add</&>)</i>
 %}
 %foreach my $addr ( @one_time_Ccs ) {
+<span style="white-space: nowrap;">
 <input
     id="UpdateBcc-<%$addr%>"
     name="UpdateBcc-<%$addr%>"
@@ -80,6 +83,7 @@
     onClick="checkboxToInput('UpdateBcc', <% "UpdateBcc-$addr" |n,j%>, <%$clean_addr|n,j%> );"
     <% $ARGS{'UpdateBcc-'.$addr} ? 'checked="checked"' : ''%> >
       <label for="UpdateBcc-<%$addr%>"><& /Elements/ShowUser, Address => $txn_addresses{$addr}&></label>
+</span>
 %}
 </td></tr>
 <%args>

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


More information about the rt-commit mailing list