From 01219f1c480d30184d9ca39501eddfb241d58497 Mon Sep 17 00:00:00 2001
From: iam4x <iam4x@me.com>
Date: Mon, 27 Mar 2017 11:08:09 +0200
Subject: [PATCH] feat(connectors): `sortBy` to `['isRefined', 'count:desc']`

---
 src/connectors/hierarchical-menu/connectHierarchicalMenu.js | 6 +++---
 src/connectors/menu/connectMenu.js                          | 6 +++---
 src/connectors/refinement-list/connectRefinementList.js     | 6 +++---
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/connectors/hierarchical-menu/connectHierarchicalMenu.js b/src/connectors/hierarchical-menu/connectHierarchicalMenu.js
index 8d799c380d..fd8f7fac3e 100644
--- a/src/connectors/hierarchical-menu/connectHierarchicalMenu.js
+++ b/src/connectors/hierarchical-menu/connectHierarchicalMenu.js
@@ -16,7 +16,7 @@ search.addWidget(
     [ rootPath = null ],
     [ showParentLevel = true ],
     [ limit = 10 ],
-    [ sortBy = ['name:asc'] ],
+    [ sortBy = ['isRefined', 'count:desc'] ],
   });
 );
 Full documentation available at https://community.algolia.com/instantsearch.js/connectors/connectHierarchicalMenu.html
@@ -29,7 +29,7 @@ Full documentation available at https://community.algolia.com/instantsearch.js/c
  * @param  {string} [rootPath] Prefix path to use if the first level is not the root level.
  * @param  {string} [showParentLevel=false] Show the parent level of the current refined value
  * @param  {number} [limit=10] How much facet values to get [*]
- * @param  {string[]|Function} [sortBy=['name:asc']] How to sort refinements. Possible values: `count|isRefined|name:asc|name:desc`.
+ * @param  {string[]|Function} [sortBy=['isRefined', 'count:desc']] How to sort refinements. Possible values: `count|isRefined|name:asc|name:desc`.
  *   You can also use a sort function that behaves like the standard Javascript [compareFunction](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#Syntax).
  */
 
@@ -57,7 +57,7 @@ export default function connectHierarchicalMenu(renderFn) {
       rootPath = null,
       showParentLevel = true,
       limit = 10,
-      sortBy = ['name:asc'],
+      sortBy = ['isRefined', 'count:desc'],
     } = widgetParams;
 
     if (!attributes || !attributes.length) {
diff --git a/src/connectors/menu/connectMenu.js b/src/connectors/menu/connectMenu.js
index 99234544b9..9b810b4660 100644
--- a/src/connectors/menu/connectMenu.js
+++ b/src/connectors/menu/connectMenu.js
@@ -16,7 +16,7 @@ search.addWidget(
   customMenu({
     attributeName,
     [ limit ],
-    [ sortBy = ['count:desc', 'name:asc'] ]
+    [ sortBy = ['isRefined', 'count:desc'] ]
   });
 );
 Full documentation available at https://community.algolia.com/instantsearch.js/connectors/connectMenu.html
@@ -26,7 +26,7 @@ Full documentation available at https://community.algolia.com/instantsearch.js/c
  * @typedef {Object} CustomMenuWidgetOptions
  * @param {string} attributeName Name of the attribute for faceting (eg. "free_shipping")
  * @param {number} [limit = 10] How many facets values to retrieve [*]
- * @param {string[]|function} [sortBy = ['count:desc', 'name:asc']] How to sort refinements. Possible values: `count|isRefined|name:asc|name:desc`.
+ * @param {string[]|function} [sortBy = ['isRefined', 'count:desc']] How to sort refinements. Possible values: `count|isRefined|name:asc|name:desc`.
  *   You can also use a sort function that behaves like the standard Javascript [compareFunction](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#Syntax). [*]
  */
 
@@ -52,7 +52,7 @@ export default function connectMenu(renderFn) {
     const {
       attributeName,
       limit = 10,
-      sortBy = ['count:desc', 'name:asc'],
+      sortBy = ['isRefined', 'count:desc'],
     } = widgetParams;
 
     if (!attributeName) {
diff --git a/src/connectors/refinement-list/connectRefinementList.js b/src/connectors/refinement-list/connectRefinementList.js
index 42e794b238..4dc4a86925 100644
--- a/src/connectors/refinement-list/connectRefinementList.js
+++ b/src/connectors/refinement-list/connectRefinementList.js
@@ -18,7 +18,7 @@ search.addWidget(
     attributeName,
     [ operator = 'or' ],
     [ limit ],
-    [ sortBy = ['count:desc', 'name:asc'] ]
+    [ sortBy = ['isRefined', 'count:desc'] ]
   });
 );
 Full documentation available at https://community.algolia.com/instantsearch.js/connectors/connectRefinementList.html
@@ -38,7 +38,7 @@ export const checkUsage = ({attributeName, operator, usageMessage}) => {
  * @property {string} attributeName the attribute in the records that are used by the widget
  * @property {string} operator how the filters are combined together
  * @property {number} limit the max number of items displayed
- * @property {string[]} sortBy how the values are ordered
+ * @param {string[]|function} [sortBy = ['isRefined', 'count:desc']] How to sort refinements. Possible values: `count|isRefined|name:asc|name:desc`.
  */
 
 /**
@@ -68,7 +68,7 @@ export default function connectRefinementList(renderFn) {
       attributeName,
       operator = 'or',
       limit,
-      sortBy = ['count:desc', 'name:asc'],
+      sortBy = ['isRefined', 'count:desc'],
     } = widgetParams;
 
     checkUsage({attributeName, operator, usage});