Don't print • divider before blank platform field (#420)

iftechfoundation · Sep 2, 2024 · c7f4e69 · c7f4e69
1 parent de7eb0e
commit c7f4e69
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 8 deletions.
diff --git a/IFComp/root/src/_current_entry_row.tt b/IFComp/root/src/_current_entry_row.tt
@@ -49,20 +49,29 @@
 
         <p><em>
         [% IF entry.genre %]
-        <span itemprop="genre">[% entry.genre.ucfirst | html %]</span> •
+        <span itemprop="genre">[% entry.genre.ucfirst | html %]</span>
         [% END %]
 
         [% IF entry.playtime %]
-        <span itemprop="size">[% entry.playtime.ucfirst %]</span> •
+        [% IF entry.genre %]
+        •
+        [% END %]
+        <span itemprop="size">[% entry.playtime.ucfirst %]</span>
         [% END %]
 
+        [% IF entry.style && ( entry.genre || entry.playtime ) %]
+        •
+        [% END %]
         [% IF entry.style == 'parser' %]
-        <span itemprop="interactivityType">Parser-based</span> •
+        <span itemprop="interactivityType">Parser-based</span>
         [% ELSIF entry.style == 'choice' %]
-        <span itemprop="interactivityType">Choice-based</span> •
+        <span itemprop="interactivityType">Choice-based</span>
         [% END %]
 
         [% UNLESS entry.platform == 'other' %]
+        [% IF entry.genre || entry.playtime || entry.style %]
+        •
+        [% END %]
         [% IF entry.platform == 'parchment' || entry.platform == 'inform-website' || entry.platform == 'inform' || entry.platform == 'quixe' || entry.platform == 'quixe2' %]
             [% IF entry.is_zcode %]
             <span itemprop="gamePlatform">Z-code</span> [% INCLUDE guide_link %]

diff --git a/IFComp/root/src/comp/cover_sheet.tt b/IFComp/root/src/comp/cover_sheet.tt
@@ -49,20 +49,29 @@ judging on [% comp.judging_begins.strftime( '%{month_name} %{day}' ) %], [% comp
 
         <p><em>
         [% IF entry.genre %]
-        [% entry.genre.ucfirst | html %] •
+        [% entry.genre.ucfirst | html %]
         [% END %]
 
         [% IF entry.playtime %]
-        [% entry.playtime.ucfirst %] •
+        [% IF entry.genre %]
+        •
+        [% END %]
+        [% entry.playtime.ucfirst %]
         [% END %]
 
+        [% IF entry.style && ( entry.genre || entry.playtime ) %]
+        •
+        [% END %]
         [% IF entry.style == 'parser' %]
-        Parser-based •
+        Parser-based
         [% ELSIF entry.style == 'choice' %]
-        Choice-based •
+        Choice-based
         [% END %]
 
         [% UNLESS entry.platform == 'other' %]
+        [% IF entry.genre || entry.playtime || entry.style %]
+        •
+        [% END %]
         [% IF entry.platform == 'parchment' || entry.platform == 'inform-website' || entry.platform == 'inform' || entry.platform == 'quixe' || entry.platform == 'quixe2' %]
             [% IF entry.is_zcode %]
             Z-code