From f832dac5c6afafcc766b8f09277afdacc75d3bdc Mon Sep 17 00:00:00 2001
From: Zarino Zappia <mail@zarino.co.uk>
Date: Wed, 10 Mar 2021 17:49:39 +0000
Subject: [PATCH] Standardise use of .business-section__secondary__item

Ensuring that each item in the business section sidebar is wrapped in
a .business-section__secondary__item allows us to simplify the vertical
spacing styles, and prevents awkward double space above calendars.
---
 www/docs/style/sass/pages/_business.scss      |  9 +++---
 .../html/section/_business_section.php        | 30 +++++++++++--------
 .../templates/html/section/day.php            | 10 ++++---
 3 files changed, 29 insertions(+), 20 deletions(-)

diff --git a/www/docs/style/sass/pages/_business.scss b/www/docs/style/sass/pages/_business.scss
index 7118e795b9..f6e36b33bb 100644
--- a/www/docs/style/sass/pages/_business.scss
+++ b/www/docs/style/sass/pages/_business.scss
@@ -97,9 +97,12 @@
     }
 
     .rss-feed {
-        margin-top: 2em;
         font-size: 0.8em;
     }
+
+    .calendar {
+        margin: 1em 0 2em 0;
+    }
 }
 
 .business-section__secondary__item {
@@ -114,9 +117,7 @@
     }
 }
 
-.business-section__secondary__item + .business-section__secondary__item,
-.business-section__secondary .calendar,
-.business-section__secondary .minisurvey {
+.business-section__secondary__item + .business-section__secondary__item {
     margin-top: 2em;
 
     @media (min-height: 700px) {
diff --git a/www/includes/easyparliament/templates/html/section/_business_section.php b/www/includes/easyparliament/templates/html/section/_business_section.php
index 8d4efb0f42..dd33144b79 100644
--- a/www/includes/easyparliament/templates/html/section/_business_section.php
+++ b/www/includes/easyparliament/templates/html/section/_business_section.php
@@ -14,18 +14,24 @@
                         <h3>What is this?</h3>
                         <?php include '_' . $section . '_desc.php'; ?>
                     </div>
-                <?php
-                $calendar = $content['calendar']['years'];
-                include '_calendar_section.php';
-                if ( !isset($no_survey) ) {
-                    include( dirname(__FILE__) . '/../sidebar/looking_for.php' );
-                }
-                ?>
-                <?php if ( isset($content['rssurl']) ) { ?>
-                <p class="rss-feed">
-                    <a href="<?= WEBPATH . $content['rssurl'] ?>">RSS feed of <?= $title ?></a>
-                </p>
-                <?php } ?>
+                    <div class="business-section__secondary__item">
+                        <?php
+                        $calendar = $content['calendar']['years'];
+                        include '_calendar_section.php';
+                        ?>
+                    </div>
+                  <?php if ( !isset($no_survey) ) { ?>
+                    <div class="business-section__secondary__item">
+                        <?php include( dirname(__FILE__) . '/../sidebar/looking_for.php' ); ?>
+                    </div>
+                  <?php } ?>
+                  <?php if ( isset($content['rssurl']) ) { ?>
+                    <div class="business-section__secondary__item">
+                        <p class="rss-feed">
+                            <a href="<?= WEBPATH . $content['rssurl'] ?>">RSS feed of <?= $title ?></a>
+                        </p>
+                    </div>
+                  <?php } ?>
                 </div>
             <?php } else { ?>
              <div class="business-section__primary">
diff --git a/www/includes/easyparliament/templates/html/section/day.php b/www/includes/easyparliament/templates/html/section/day.php
index 45a80e77bc..19564c432e 100644
--- a/www/includes/easyparliament/templates/html/section/day.php
+++ b/www/includes/easyparliament/templates/html/section/day.php
@@ -72,10 +72,12 @@
                 <h3>What is this?</h3>
                 <?php include '_' . $section . '_desc.php'; ?>
             </div>
-            <?php
-                include '_calendar_section.php';
-                include( dirname(__FILE__) . '/../sidebar/looking_for.php' );
-            ?>
+            <div class="business-section__secondary__item">
+                <?php include '_calendar_section.php'; ?>
+            </div>
+            <div class="business-section__secondary__item">
+                <?php include( dirname(__FILE__) . '/../sidebar/looking_for.php' ); ?>
+            </div>
         </div>
       <?php } else { ?>
         <div class="business-section__primary">