From 9e9d438b0e4731e1511c91e3e7124fc7688e1dc7 Mon Sep 17 00:00:00 2001
From: Pixelastic <tim@pixelastic.com>
Date: Fri, 16 Oct 2015 15:58:26 +0200
Subject: [PATCH] feat(headerFooter): Add BEM classes to header and footer

All headers and footers will now receive a `ais-header` and
`ais-footer` class. Additional classes can also be added through
`cssClasses.{root,header,body,footer}`.

It means that it is no longer necessary to pass a full HTML div in the
`templates.header`. One can simply pass the header text content and
pass a `cssClasses.header` custom class.

So far only the `refinementList` widgets correctly add the
`ais-widgetname-header`/`ais-widgetname-footer` classes. I'll update
the other widgets once this gets merged.

Fixes #259
Relies on https://github.com/algolia/instantsearch.js/pull/264 being
merged first
---
 README.md                                  | 30 +++++++++++--------
 decorators/headerFooter.js                 | 35 ++++++++++++++--------
 example/app.js                             | 20 +++++++++----
 widgets/refinement-list/refinement-list.js |  4 +++
 4 files changed, 59 insertions(+), 30 deletions(-)

diff --git a/README.md b/README.md
index abb04d2caa..58398036ab 100644
--- a/README.md
+++ b/README.md
@@ -593,19 +593,25 @@ search.addWidget(
 #### Styling
 
 ```html
-<div class="ais-refinement-list--list">
-  <div class="ais-refinement-list--item">
-    <label class="ais-refinement-list--label">
-      <input type="checkbox" class="ais-refinement-list--checkbox" value="your_value"> Your value
-      <span class="ais-refinement-list--count">42</span>
-    </label>
-  </div>
-  <div class="ais-refinement-list--item ais-refinement-list--item__active">
-    <label class="ais-refinement-list--label">
-      <input type="checkbox" class="ais-refinement-list--checkbox" value="your_selected_value" checked="checked"> Your selected value
-      <span class="ais-refinement-list--count">42</span>
-    </label>
+<div class="ais-refinement-list">
+  <div class="ais-refinement-list--header ais-header">[custom header template]</div>
+  <div class="ais-refinement-list--body">
+    <div class="ais-refinement-list--list">
+      <div class="ais-refinement-list--item">
+        <label class="ais-refinement-list--label">
+          <input type="checkbox" class="ais-refinement-list--checkbox" value="your_value"> Your value
+          <span class="ais-refinement-list--count">42</span>
+        </label>
+      </div>
+      <div class="ais-refinement-list--item ais-refinement-list--item__active">
+        <label class="ais-refinement-list--label">
+          <input type="checkbox" class="ais-refinement-list--checkbox" value="your_selected_value" checked="checked"> Your selected value
+          <span class="ais-refinement-list--count">42</span>
+        </label>
+      </div>
+    </div>
   </div>
+  <div class="ais-refinement-list--footer ais-footer">[custom footer template]</div>
 </div>
 ```
 
diff --git a/decorators/headerFooter.js b/decorators/headerFooter.js
index 171f779af2..d002b34692 100644
--- a/decorators/headerFooter.js
+++ b/decorators/headerFooter.js
@@ -1,6 +1,6 @@
 var React = require('react');
 
-var cx = require('classnames');
+var cx = require('classnames/dedupe');
 
 var Template = require('../components/Template');
 
@@ -10,12 +10,25 @@ function headerFooter(ComposedComponent) {
       // override potential widget's defined transformData,
       // header and footer currently do not have it
       var transformData = null;
+      var templateProps = this.props.templateProps;
+      var classNames = {
+        root: this.props.cssClasses.root,
+        header: cx(this.props.cssClasses.header, 'ais-header'),
+        body: this.props.cssClasses.body,
+        footer: cx(this.props.cssClasses.footer, 'ais-footer')
+      };
 
       return (
-        <div className={cx(this.props.cssClasses.root)}>
-          <Template templateKey="header" {...this.props.templateProps} transformData={transformData} />
-          <ComposedComponent {...this.props} />
-          <Template templateKey="footer" {...this.props.templateProps} transformData={transformData} />
+        <div className={classNames.root}>
+          <div className={classNames.header}>
+            <Template templateKey="header" {...templateProps} transformData={transformData} />
+          </div>
+          <div className={classNames.body}>
+            <ComposedComponent {...this.props} />
+          </div>
+          <div className={classNames.footer}>
+            <Template templateKey="footer" {...templateProps} transformData={transformData} />
+          </div>
         </div>
       );
     }
@@ -23,17 +36,15 @@ function headerFooter(ComposedComponent) {
 
   HeaderFooter.propTypes = {
     cssClasses: React.PropTypes.shape({
-      root: React.PropTypes.oneOfType([
-        React.PropTypes.string,
-        React.PropTypes.arrayOf(React.PropTypes.string)
-      ])
+      root: React.PropTypes.string,
+      header: React.PropTypes.string,
+      body: React.PropTypes.string,
+      footer: React.PropTypes.string
     })
   };
 
   HeaderFooter.defaultProps = {
-    cssClasses: {
-      root: null
-    }
+    cssClasses: {}
   };
 
   // precise displayName for ease of debugging (react dev tool, react warnings)
diff --git a/example/app.js b/example/app.js
index 6f2aa8bde7..65684218d9 100644
--- a/example/app.js
+++ b/example/app.js
@@ -70,12 +70,13 @@ search.addWidget(
     operator: 'or',
     limit: 10,
     cssClasses: {
+      header: 'panel-heading',
       list: 'nav nav-stacked panel-body',
       item: 'checkbox',
       count: 'badge pull-right'
     },
     templates: {
-      header: '<div class="panel-heading">Brands</div>'
+      header: 'Brands'
     }
   })
 );
@@ -87,10 +88,11 @@ search.addWidget(
     operator: 'and',
     limit: 10,
     cssClasses: {
+      header: 'panel-heading',
       root: 'list-group'
     },
     templates: {
-      header: '<div class="panel-heading">Price ranges</div>',
+      header: 'Price ranges',
       item: require('./templates/and.html')
     },
     transformData: function(data) {
@@ -105,8 +107,11 @@ search.addWidget(
     container: '#free-shipping',
     facetName: 'free_shipping',
     label: 'Free Shipping',
+    cssClasses: {
+      header: 'panel-heading'
+    },
     templates: {
-      header: '<div class="panel-heading">Shipping</div>',
+      header: 'Shipping',
       body: require('./templates/free-shipping.html')
     }
   })
@@ -118,10 +123,11 @@ search.addWidget(
     facetName: 'categories',
     limit: 10,
     cssClasses: {
+      header: 'panel-heading',
       root: 'list-group'
     },
     templates: {
-      header: '<div class="panel-heading">Categories</div>',
+      header: 'Categories',
       item: require('./templates/category.html')
     }
   })
@@ -132,10 +138,11 @@ search.addWidget(
     container: '#price',
     facetName: 'price',
     cssClasses: {
+      header: 'panel-heading',
       body: 'panel-body'
     },
     templates: {
-      header: '<div class="panel-heading">Price</div>'
+      header: 'Price'
     },
     tooltips: {
       format: function(formattedValue) {
@@ -151,10 +158,11 @@ search.addWidget(
     attributes: ['hierarchicalCategories.lvl0', 'hierarchicalCategories.lvl1', 'hierarchicalCategories.lvl2'],
     cssClasses: {
       root: 'list-group',
+      header: 'panel-heading',
       list: 'hierarchical-categories-list'
     },
     templates: {
-      header: '<div class="panel-heading">Hierarchical categories</div>',
+      header: 'Hierarchical categories',
       item: require('./templates/category.html')
     }
   })
diff --git a/widgets/refinement-list/refinement-list.js b/widgets/refinement-list/refinement-list.js
index ced7d86004..555849af20 100644
--- a/widgets/refinement-list/refinement-list.js
+++ b/widgets/refinement-list/refinement-list.js
@@ -87,6 +87,10 @@ function refinementList({
       var facetValues = results.getFacetValues(facetName, {sortBy: sortBy}).slice(0, limit);
 
       cssClasses = {
+        root: cx(bem(null), cssClasses.root),
+        header: cx(bem('header'), cssClasses.header),
+        body: cx(bem('body'), cssClasses.body),
+        footer: cx(bem('footer'), cssClasses.footer),
         list: cx(bem('list'), cssClasses.list),
         item: cx(bem('item'), cssClasses.item),
         active: cx(bem('item', 'active'), cssClasses.active),