Ommit header markup if not content is passed

Author: dfmcphee

src/components/Page/Page.scss

@@ -57,19 +57,14 @@ html, body {
 .Header-hasPagination,
 .Header-hasBreadcrumbs {
   padding-top: spacing(loose);
-  padding-bottom: rem(24px);
-
-  &.Header-hasSecondaryActions {
-    padding-bottom: spacing(loose);
-  }
 }
 
 .Header-hasSecondaryActions {
   padding-top: rem(24px);
-  padding-bottom: spacing(loose);
 }
 
 .Header-hasSeparator {
+  padding-bottom: spacing();
   border-bottom: border();
 
   + .Content {

src/components/Page/Page.tsx

@@ -53,7 +53,7 @@ export default class Page extends React.PureComponent<Props, never> {
       fullWidth && styles.fullWidth,
     );
 
-    const headerMarkup = this.context.easdk
+    const headerMarkup = this.context.easdk || !this.hasHeaderContent()
       ? null
       : <Header {...rest} />;
 
@@ -74,4 +74,14 @@ export default class Page extends React.PureComponent<Props, never> {
       easdk.Bar.update(this.props);
     }
   }
+
+  private hasHeaderContent() {
+    const { title, primaryAction, secondaryActions, breadcrumbs } = this.props;
+    return (
+      (title && title !== '')
+      || primaryAction
+      || (secondaryActions && secondaryActions.length > 0)
+      || (breadcrumbs && breadcrumbs.length > 0)
+    );
+  }
 }