From f2be88e7050bba910756ba77c8ec16951495a147 Mon Sep 17 00:00:00 2001
From: Monish Deb <deb.monish@gmail.com>
Date: Tue, 29 Oct 2019 19:05:40 +0530
Subject: [PATCH] core#1346: Tagsets should display vertically in activity/new
 case forms

---
 templates/CRM/Activity/Form/Activity.tpl |  4 +-
 templates/CRM/Case/Form/Case.tpl         |  4 +-
 templates/CRM/common/Tagset.tpl          | 54 ++++++++++++++----------
 3 files changed, 38 insertions(+), 24 deletions(-)

diff --git a/templates/CRM/Activity/Form/Activity.tpl b/templates/CRM/Activity/Form/Activity.tpl
index ac19690e42e3..9e67b47e5554 100644
--- a/templates/CRM/Activity/Form/Activity.tpl
+++ b/templates/CRM/Activity/Form/Activity.tpl
@@ -184,7 +184,9 @@
   {/if}
 
   {if $tagsetInfo.activity}
-  <tr class="crm-activity-form-block-tag_set">{include file="CRM/common/Tagset.tpl" tagsetType='activity' tableLayout=true}</tr>
+    <tr class="crm-activity-form-block-tag_set">
+      {include file="CRM/common/Tagset.tpl" tagsetType='activity' tableLayout=true}
+    </tr>
   {/if}
 
   {if $action neq 4 OR $viewCustomData}
diff --git a/templates/CRM/Case/Form/Case.tpl b/templates/CRM/Case/Form/Case.tpl
index fb000fbe7938..e512029e5337 100644
--- a/templates/CRM/Case/Form/Case.tpl
+++ b/templates/CRM/Case/Form/Case.tpl
@@ -119,7 +119,9 @@
     </td>
 </tr>
 
-<tr class="crm-case-form-block-tag_set"><td colspan="2">{include file="CRM/common/Tagset.tpl" tagsetType='case'}</td></tr>
+<tr class="crm-case-form-block-tag_set">
+    {include file="CRM/common/Tagset.tpl" tagsetType='case' tableLayout=true}
+</tr>
 
 </table>
 {/if}
diff --git a/templates/CRM/common/Tagset.tpl b/templates/CRM/common/Tagset.tpl
index 83e781223e01..278b9d5cf3b3 100644
--- a/templates/CRM/common/Tagset.tpl
+++ b/templates/CRM/common/Tagset.tpl
@@ -26,28 +26,38 @@
 {if empty($tagsetType)}
   {assign var="tagsetType" value="contact"}
 {/if}
-{foreach from=$tagsetInfo.$tagsetType item=tagset}
-  {assign var="elemName" value=$tagset.tagsetElementName}
-  {if empty($tagsetElementName) or $tagsetElementName eq $elemName}
-    {assign var="parID" value=$tagset.parentID}
-    {assign var="skipEntityAction" value=$tagset.skipEntityAction}
-    {if $tableLayout}
-      <td class="label">
-        {$form.$elemName.$parID.label}
-      </td>
-      <td class="{$tagsetType}-tagset {$tagsetType}-tagset-{$tagset.parentID}-section">
-          {$form.$elemName.$parID.html}
-      </td>
-    {else}
-      <div class="crm-section tag-section {$tagsetType}-tagset {$tagsetType}-tagset-{$tagset.parentID}-section">
-        <div class="crm-clearfix">
-          {$form.$elemName.$parID.label}
-          {$form.$elemName.$parID.html}
-        </div>
-      </div>
-    {/if}
-  {/if}
-{/foreach}
+{if $tableLayout}
+  <td colspan="2" class="crm-content-block">
+    <table>
+{/if}
+    {foreach from=$tagsetInfo.$tagsetType item=tagset}
+      {assign var="elemName" value=$tagset.tagsetElementName}
+      {if empty($tagsetElementName) or $tagsetElementName eq $elemName}
+        {assign var="parID" value=$tagset.parentID}
+        {assign var="skipEntityAction" value=$tagset.skipEntityAction}
+        {if $tableLayout}
+          <tr>
+            <td class="label">
+              {$form.$elemName.$parID.label}
+            </td>
+            <td class="{$tagsetType}-tagset {$tagsetType}-tagset-{$tagset.parentID}-section">
+              {$form.$elemName.$parID.html}
+            </td>
+          </tr>
+        {else}
+          <div class="crm-section tag-section {$tagsetType}-tagset {$tagsetType}-tagset-{$tagset.parentID}-section">
+            <div class="crm-clearfix">
+              {$form.$elemName.$parID.label}
+              {$form.$elemName.$parID.html}
+            </div>
+          </div>
+        {/if}
+      {/if}
+    {/foreach}
+{if $tableLayout}
+    </table>
+  </td>
+{/if}
 
 {if !$skipEntityAction and empty($form.frozen)}
   <script type="text/javascript">