Hide line break if event has no location

Author: AaronC81

static/js/calendar.js

@@ -244,6 +244,14 @@ function displayInfoPanel(target, event) {
         `${event.when_human.start_time}–${event.when_human.end_time} ${event.when_human.long_start_date}`
     document.getElementById('calendar-info-where').innerText = event.location;
 
+    // If there's no "where", hide the line break between "when" and "where"
+    // so we don't have a useless break
+    if (!event.location || event.location.trim() == '') {
+        document.getElementById('calendar-info-br').style.display = 'none';
+    } else {
+        document.getElementById('calendar-info-br').style.display = 'initial';
+    }
+
     const desc = document.getElementById('calendar-info-description');
     desc.innerHTML = event.description;
     while (desc.firstChild.nodeName.toLowerCase() == "br") {

templates/calendar.handlebars

@@ -48,7 +48,7 @@
       <a href="javascript:closeInfoPanel()" aria-label="Close Event Info Panel" id="calendar-info-close">&times;</a>
       <h3 id="calendar-info-title"></h4>
       <b id="calendar-info-when"></b>
-      <br>
+      <br id="calendar-info-br">
       <b id="calendar-info-where"></b>
       <a href="#" id="calendar-join-button" class="hidden">Join Google Meet</a>
       <div id="calendar-info-description"></div>