[Rt-commit] rt branch, 5.0/fix-selfservice-custom-field-layout, updated. rt-5.0.0beta1-45-g87917f6579

Aaron Trevena ast at bestpractical.com
Fri Jul 3 10:06:02 EDT 2020


The branch, 5.0/fix-selfservice-custom-field-layout has been updated
       via  87917f6579b5691718a7d60bdce0d5b20df7adfa (commit)
       via  1fba4283be187ed742accbe4336f224d046c4725 (commit)
      from  7b67916fb98b2e6e6c22bd8ab4126323b44adeb1 (commit)

Summary of changes:
 share/html/Articles/Article/Elements/EditCustomFields  | 2 +-
 share/html/Elements/EditCustomFields                   | 7 ++++---
 share/html/SelfService/Create.html                     | 3 ++-
 share/html/Ticket/Elements/EditTransactionCustomFields | 2 +-
 share/static/css/elevator-light/misc.css               | 5 -----
 share/static/css/elevator-light/ticket-form.css        | 6 ------
 share/static/css/mobile.css                            | 4 ++--
 7 files changed, 10 insertions(+), 19 deletions(-)

- Log -----------------------------------------------------------------
commit 1fba4283be187ed742accbe4336f224d046c4725
Author: Aaron Trevena <ast at bestpractical.com>
Date:   Fri Jul 3 14:53:26 2020 +0100

    Simplify custom field column class logic

diff --git a/share/html/Elements/EditCustomFields b/share/html/Elements/EditCustomFields
index f47b348968..7a0ba1f27a 100644
--- a/share/html/Elements/EditCustomFields
+++ b/share/html/Elements/EditCustomFields
@@ -55,13 +55,13 @@
 % my $Type = $CustomField->Type || 'Unknown';
 
   <div class="form-row edit-custom-field cftype-<% $Type %>">
-    <div class="cflabel label col-<% ($InSidebar) ? 3 : 2 %>">
+    <div class="cflabel label col-<% $LabelCols %>">
       <% $CustomField->Name %>:
 % if ( $CustomField->EntryHint ) {
       <span class="far fa-question-circle icon-helper" data-toggle="tooltip" data-placement="top" data-original-title="<% $CustomField->EntryHint %>"></span>
 % }
     </div>
-    <div class="value col-<% ($InSidebar) ? 9 : 8 %>">
+    <div class="value col-<% $ValueCols %>">
 % my $default = $m->notes('Field-' . $CustomField->Id);
 % $default ||= $ARGS{"CustomField-". $CustomField->Id };
       <& /Elements/EditCustomField,
@@ -113,7 +113,8 @@ $Object
 $CustomFields => undef
 $Grouping     => undef
 $InTable => 0
-$InSidebar => 1
+$LabelCols => 3
+$ValueCols => 9
 $ShowHints => 1
 $ForCreation => 0
 </%ARGS>
diff --git a/share/html/SelfService/Create.html b/share/html/SelfService/Create.html
index 9e3f32f679..ac480ffae2 100644
--- a/share/html/SelfService/Create.html
+++ b/share/html/SelfService/Create.html
@@ -98,7 +98,8 @@
     CustomFields    => $queue_obj->TicketCustomFields,
     InTable         => 1,
     ForCreation     => 1,
-    InSidebar       => 0
+    LabelCols       => 2,
+    ValueCols       => 8,
     &>
 </div>
 </div>

commit 87917f6579b5691718a7d60bdce0d5b20df7adfa
Author: Aaron Trevena <ast at bestpractical.com>
Date:   Fri Jul 3 15:04:54 2020 +0100

    Drop cflabel style from custom field forms
    
    Remove the cflabel style from custom field forms and css

diff --git a/share/html/Articles/Article/Elements/EditCustomFields b/share/html/Articles/Article/Elements/EditCustomFields
index 10161c66b1..5582623a84 100644
--- a/share/html/Articles/Article/Elements/EditCustomFields
+++ b/share/html/Articles/Article/Elements/EditCustomFields
@@ -47,7 +47,7 @@
 %# END BPS TAGGED BLOCK }}}
 % while (my $CustomField = $CustomFields->Next()) {
 <div class="form-row">
-  <div class="cflabel label col-3">
+  <div class="label col-3">
     <span class="name"><%$CustomField->Name%>:</span>
 % if ( $CustomField->EntryHint ) {
     <span class="far fa-question-circle icon-helper" data-toggle="tooltip" data-placement="top" data-original-title="<% $CustomField->EntryHint %>"></span>
diff --git a/share/html/Elements/EditCustomFields b/share/html/Elements/EditCustomFields
index 7a0ba1f27a..97dd8aa4f5 100644
--- a/share/html/Elements/EditCustomFields
+++ b/share/html/Elements/EditCustomFields
@@ -55,7 +55,7 @@
 % my $Type = $CustomField->Type || 'Unknown';
 
   <div class="form-row edit-custom-field cftype-<% $Type %>">
-    <div class="cflabel label col-<% $LabelCols %>">
+    <div class="label col-<% $LabelCols %>">
       <% $CustomField->Name %>:
 % if ( $CustomField->EntryHint ) {
       <span class="far fa-question-circle icon-helper" data-toggle="tooltip" data-placement="top" data-original-title="<% $CustomField->EntryHint %>"></span>
diff --git a/share/html/Ticket/Elements/EditTransactionCustomFields b/share/html/Ticket/Elements/EditTransactionCustomFields
index fc87792f54..c5d6886259 100644
--- a/share/html/Ticket/Elements/EditTransactionCustomFields
+++ b/share/html/Ticket/Elements/EditTransactionCustomFields
@@ -53,7 +53,7 @@
 % while (my $CF = $CustomFields->Next()) {
 % next unless $CF->CurrentUserHasRight('ModifyCustomField');
 <div class="form-row">
-<div class="label cflabel col-3">
+<div class="label col-3">
   <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>
diff --git a/share/static/css/elevator-light/misc.css b/share/static/css/elevator-light/misc.css
index f37e36834f..0a8235f95a 100644
--- a/share/static/css/elevator-light/misc.css
+++ b/share/static/css/elevator-light/misc.css
@@ -28,11 +28,6 @@ hr.clear {
     font-size: 1px;
 }
 
-/* Custom fields */
-
-.cflabel .name { font-weight: bold; }
-.cflabel .type { font-style: italic; }
-
 .query-stacktrace-toggle {
     float: right;
 }
diff --git a/share/static/css/elevator-light/ticket-form.css b/share/static/css/elevator-light/ticket-form.css
index 26bb0ffb1e..c2ceb12a2e 100644
--- a/share/static/css/elevator-light/ticket-form.css
+++ b/share/static/css/elevator-light/ticket-form.css
@@ -1,9 +1,3 @@
-#ticket-create-metadata .cflabel .name,
-#ticket-update-metadata .cflabel .name,
-#ticket-info-basics .cflabel .name {
-    font-weight: normal;
-}
-
 .messagedetails input[type="text"],
 .messagedetails input[type="email"] {
     -webkit-box-sizing: border-box;
diff --git a/share/static/css/mobile.css b/share/static/css/mobile.css
index 831e67186f..78f5d06c89 100644
--- a/share/static/css/mobile.css
+++ b/share/static/css/mobile.css
@@ -106,7 +106,7 @@ hr.clear {
 }
 
 
-.label, .labeltop, .cflabel {
+.label, .labeltop {
     font-weight: normal;
 }
 .value { 
@@ -408,7 +408,7 @@ div.txn-content {
 
 }
 
-.label, .cflabel {
+.label {
     text-align: left;
     width: 10em;
     color: #666;

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


More information about the rt-commit mailing list