Update views to use dl format introduced in 4.x

UIUCLibrary · Apr 13, 2023 · 4f975e3 · 4f975e3
1 parent f2e4101
commit 4f975e3
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 20 deletions.
diff --git a/view/teams/index/role-detail.phtml b/view/teams/index/role-detail.phtml
@@ -3,6 +3,7 @@ $role = $this->role;
 $team_users = $this->team_users;
 $translate = $this->plugin('translate');
 $escape = $this->plugin('escapeHtml');
+$this->htmlElement('body')->appendAttribute('class', 'items show');
 
 $sectionNavs = [
     'item-metadata' => $translate('Info'),
@@ -21,14 +22,20 @@ $sectionNavs = [
 </div>
 
 <div id="item-metadata" class="active section">
-    <div class="meta-group">
-        <h4><?php echo $translate('Name'); ?></h4>
-        <div class="value"><?php echo $escape($role->getName()); ?></div>
-    </div>
-    <div class="meta-group">
-        <h4><?php echo $translate('Comment'); ?></h4>
-        <div class="value"><?php echo $escape($role->getComment()); ?></div>
-    </div>
+    <dl>
+        <div class="property">
+            <dt><?php echo $translate('Name'); ?></dt>
+            <dd class="value">
+                <span class="value-content"><?php echo $escape($role->getName()); ?></span>
+            </dd>
+        </div>
+        <div class="property">
+            <dt><?php echo $translate('Description'); ?></dt>
+            <dd class="value">
+                <span class="value-content"><?php echo $escape($role->getComment()); ?></span>
+            </dd>
+        </div>
+    </dl>
 </div>
 
 <div id="item-linked" class="section">

diff --git a/view/teams/index/team-detail.phtml b/view/teams/index/team-detail.phtml
@@ -45,7 +45,6 @@ $sectionNavs = [
     <?php endif; ?>
 </div>
 <div id="item-metadata" class="active section">
-
     <dl>
         <div class="property">
             <dt><?php echo $translate('Name'); ?></dt>

diff --git a/view/teams/partial/user/view.phtml b/view/teams/partial/user/view.phtml
@@ -1,14 +1,12 @@
 
-<div class="property">
-    <h4>Teams</h4>
-    <?php         foreach ($this->team_users as $team_user):
-            ?>
-    <div class="values">
-        <?php             echo sprintf('%s (%s)<br>', $team_user->getTeam()->getName(), $team_user->getRole()->getName());
-            ?>
+<dl >
+    <div class="property">
+        <dt>Team(s)</dt>
+        <?php foreach ($this->team_users as $team_user): ?>
+        <dd class="value">
+            <?php echo sprintf('%s (%s)<br>', $team_user->getTeam()->getName(), $team_user->getRole()->getName()); ?>
+        </dd>
+        <?php endforeach; ?>
     </div>
-    <?php         endforeach;
-        ?>
-
-</div>
+</dl>