Merge pull request #89 from INN/86-hardcode-digest-sidebar

Hardcode the Digest Sidebar, style fixes for certain widgets therein

wp-content/themes/midwestenergynews/functions.php

@@ -244,3 +244,22 @@ function mwen_image_sizes() {
 	add_image_size( 'rect_thumb_half', 400, 300, true ); // smaller version of rect_thumb
 }
 add_action( 'after_setup_theme', 'mwen_image_sizes' );
+
+/**
+ * Register custom sidebars that aren't related to the homepage
+ *
+ * @link https://codex.wordpress.org/Widgetizing_Themes
+ */
+function mwen_register_sidebars() {
+	// @link https://github.com/INN/umbrella-energynewsnetwork/issues/86
+	register_sidebar( array(
+		'name' => 'Digest Sidebar',
+		'id' => 'digest-sidebar',
+		'description' => __( 'Only output on the category-digest.php template, used on the <a href="/category/digest/">Digests</a> archive.', 'mwen' ),
+		'before_widget' => '<!-- Sidebar: hardcoded digest-sidebar --><aside id="%1$s" class="%2$s clearfix">',
+		'after_widget' => '</aside>',
+		'before_title' 	=> '<h3 class="widgettitle">',
+		'after_title' 	=> '</h3>',
+	) ) ;
+}
+add_action( 'widgets_init', 'mwen_register_sidebars', 11 );

wp-content/themes/midwestenergynews/less/ninja-forms-compat.less

@@ -0,0 +1,15 @@
+// to prevent injected styles from making buttons wider than they should be
+#sidebar {
+  .field-wrap button,
+  .field-wrap input[type="button"],
+  .field-wrap input[type="submit"] {
+    width: 100%;
+    max-width: 100%;
+    box-sizing: border-box;
+  }
+}
+// Ninja forms
+.home #sidebar .widget.widget_ninja_forms_widget,
+.archive #sidebar .widget.widget_ninja_forms_widget {
+  padding-right: 24px;
+}

wp-content/themes/midwestenergynews/less/style.less

@@ -31,6 +31,7 @@
 @import "archive.less";
 @import "digest-search.less";
 @import "widgets.less";
+@import "ninja-forms-compat.less";
 @import "print.less";
 
 body.tax-region {

wp-content/themes/midwestenergynews/less/widgets.less

@@ -164,6 +164,13 @@
   }
 }
 
+// Override the media query in https://github.com/INN/largo/blob/1744bbc7ae99f501129d158c3b585cddf6b5409e/less/inc/widgets.less#L214-L222
+@media ( max-width: 768px ) {
+  #sidebar .widget {
+    padding-left: 24px;
+  }
+}
+
 /*
  * Mailchimp Newsletter Signup Widget
  */
@@ -249,28 +256,45 @@
   }
 }
 .header-ad-zone .mailchimp-signup-widget,
-.archive .widget_text,
-.home .widget_text,
-.archive .mailchimp-signup-widget,
-.home .mailchimp-signup-widget {
+.archive #sidebar .widget_text,
+.home #sidebar .widget_text,
+.archive #sidebar .mailchimp-signup-widget,
+.home #sidebar .mailchimp-signup-widget {
   border: 1px solid @greyXL;
+  .default {
+    > :first-child {
+      margin-top: 0;
+    }
+    > :last-child {
+      margin-bottom: 0;
+    }
+  }
   &.rev {
     border: none;
   }
 
   box-sizing: content-box;
-  padding: 18px 34px;
+  padding: 18px 24px;
+  // override https://github.com/INN/largo/blob/1744bbc7ae99f501129d158c3b585cddf6b5409e/less/inc/widgets.less#L214-L222
+  @media ( max-width: 768px ) {
+    padding-top: 18px;
+    padding-bottom: 18px;
+  }
   input[type=email] {
     margin: 0 auto 12px;
     display: block;
     padding: 12px;
   }
+  .btn,
   input[type=submit] {
     .btn-styles('primary');
     margin: 16px 0;
-    padding: 0.75em 2.3em;
-    width: auto;
+    padding: 0.75em 0.75em;
+    text-align: center;
+    width: 100%;
     font-size: 15px;
+    max-width: 100%;
+    box-sizing: border-box;
   }
   input[type=email] {
     width: 100%;
@@ -404,3 +428,4 @@ body .widget.saved-links {
     margin-left: 0.3em;
   }
 }
+

wp-content/themes/midwestenergynews/partials/sidebar-digest.php

@@ -0,0 +1,13 @@
+<?php
+/*
+ * For the digest category page
+ *
+ * Copied from Largo's partials/sidebar-archive.php at 0.6.4
+ * @package Largo
+ * @since 0.6.4
+ * @link https://github.com/INN/umbrella-energynewsnetwork/issues/86
+ */
+$custom_sidebar = largo_get_custom_sidebar();
+if ( !dynamic_sidebar( 'digest-sidebar' ) ) { // try custom sidebar registered in functions.php
+	// do nothing.
+}

wp-content/themes/midwestenergynews/sidebar.php

@@ -16,12 +16,15 @@
 		<?php
 			do_action('largo_before_sidebar_widgets');
 
-			if (is_archive() && !is_date())
+			if ( is_category( 'digest' ) ) {
+				get_template_part('partials/sidebar', 'digest');
+			} else if (is_archive() && !is_date()) {
 				get_template_part('partials/sidebar', 'archive');
-			else if (is_single() || is_singular())
+			} else if (is_single() || is_singular()) {
 				get_template_part('partials/sidebar', 'single');
-			else
+			} else {
 				get_template_part('partials/sidebar');
+			}
 
 			do_action('largo_after_sidebar_widgets');
 		?>