Merge pull request #920 from folio-org/STCOM-480-closedbydefault-no-w…

…orkie

STCOM-480 `<AccordionSet>` fails to acknowledge 'closedByDefault' prop of child `<Accordions>`

CHANGELOG.md

@@ -1,4 +1,7 @@
 # Change history for stripes-components
+## [5.1.2](https://github.com/folio-org/stripes-components/tree/v5.1.2) (2019-03-20)
+* `<AccordionSet>` consideres `closedByDefault` prop when setting up initial state for child accordions. Fixes STCOM-480.
+
 ## [5.1.1](https://github.com/folio-org/stripes-components/tree/v5.1.1) (2019-03-14)
 * Consider timezone in Datepicker for values containing a time offset. fixes STCOM-484.
 * Check for ref existence within MultiSelection before calling `focus()`. fixes STCOM-485.

lib/Accordion/Accordion.js

@@ -89,7 +89,7 @@ class Accordion extends React.Component {
     }
 
     if (this.props.accordionSet) {
-      this.props.accordionSet.registerAccordion(this.toggle, this.trackingId);
+      this.props.accordionSet.registerAccordion(this.toggle, this.trackingId, this.props.closedByDefault);
     }
   }
 

lib/Accordion/AccordionSet.js

@@ -100,13 +100,13 @@ class AccordionSet extends React.Component {
     this.accordionList[this.accordionList.length - 1].focus();
   }
 
-  registerAccordion(ref, id) {
+  registerAccordion(ref, id, closedByDefault = false) {
     if (indexOf(this.accordionList, ref) === -1 && ref !== null) {
       this.accordionList.push(ref);
       this.setState((curState) => {
         const newState = Object.assign({}, curState);
         if (typeof newState.expanded[id] === 'undefined') {
-          newState.expanded[id] = true;
+          newState.expanded[id] = !closedByDefault;
         }
         return newState;
       });

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@folio/stripes-components",
-  "version": "5.1.1",
+  "version": "5.1.2",
   "description": "Component library for building Stripes applications.",
   "license": "Apache-2.0",
   "repository": "folio-org/stripes-components",