fix: Wrap encounter table lists in ul

javalent · Mar 21, 2022 · d9edf7c · d9edf7c
1 parent 2727a4a
commit d9edf7c
Showing 1 changed file with 29 additions and 22 deletions.
diff --git a/src/encounter/ui/EncounterRow.svelte b/src/encounter/ui/EncounterRow.svelte
@@ -143,31 +143,35 @@
 <tr class="encounter-row">
     <td>{name}</td>
     {#if headers.includes("creatures")}
-        <td class="encounter-creatures encounter-list">
-            {#if !hide.includes("creatures") && creatures.size}
-                {#each [...creatures] as [creature, count]}
-                    <li aria-label={label(creature)}>
-                        <strong
-                            use:rollerEl={creature}
-                        />&nbsp;{creature.name}{count == 1 ? "" : "s"}
-                    </li>
-                {/each}
-            {:else}
-                -
-            {/if}
+        <td>
+            <ul class="encounter-creatures encounter-list">
+                {#if !hide.includes("creatures") && creatures.size}
+                    {#each [...creatures] as [creature, count]}
+                        <li aria-label={label(creature)}>
+                            <strong
+                                use:rollerEl={creature}
+                            />&nbsp;{creature.name}{count == 1 ? "" : "s"}
+                        </li>
+                    {/each}
+                {:else}
+                    -
+                {/if}
+            </ul>
         </td>
     {/if}
     {#if headers.includes("players")}
-        <td class="encounter-players encounter-list">
-            {#if !hide.includes("players") && players instanceof Array && players.length}
-                {#each players as player}
-                    <li>
-                        {player}
-                    </li>
-                {/each}
-            {:else}
-                -
-            {/if}
+        <td>
+            <ul class="encounter-players encounter-list">
+                {#if !hide.includes("players") && players instanceof Array && players.length}
+                    {#each players as player}
+                        <li>
+                            {player}
+                        </li>
+                    {/each}
+                {:else}
+                    -
+                {/if}
+            </ul>
         </td>
     {/if}
     {#if plugin.data.displayDifficulty}
@@ -215,4 +219,7 @@
     .icons > div:first-child :global(.clickable-icon) {
         margin-right: 0;
     }
+    ul {
+        margin: 0;
+    }
 </style>