Collapsible layer groups #66

src/components/layers/LayerList.jsx

@@ -1,4 +1,5 @@
 import React from 'react'
+import classnames from 'classnames'
 import cloneDeep from 'lodash.clonedeep'
 
 import Button from '../Button'
@@ -31,6 +32,7 @@ class LayerListContainer extends React.Component {
   constructor(props) {
     super(props)
     this.state = {
+      collapsedGroups: {},
       isOpen: {
         add: false,
       }
@@ -92,37 +94,42 @@ class LayerListContainer extends React.Component {
   }
 
   toggleLayerGroup(groupPrefix) {
-    groupedLayers().forEach(layers => {
+    const newGroups = { ...this.state.collapsedGroups }
-      if(groupPrefix === layerPrefix(layers[0].id)) {
+    if(groupPrefix in this.state.collapsedGroups) {
-        layers.forEach(layer => {
+      newGroups[groupPrefix] = !this.state.collapsedGroups[groupPrefix]
-          //HACK
+    } else {
-          //In this case it is ok to modify the metadata
+      newGroups[groupPrefix] = true
-          //because no one else depends on it
+    }
-          layer.metadata = {
+    this.setState({
-            ...layer.metadata,
+      collapsedGroups: newGroups
-            'maputnik:visible': false
-          }
-        })
-      }
     })
   }
 
   render() {
+    const isCollapsed = groupPrefix => {
+      const collapsed = this.state.collapsedGroups[groupPrefix]
+      return collapsed === undefined ? false : collapsed
+    }
+
     const listItems = []
     let idx = 0
     this.groupedLayers().forEach(layers => {
+      const groupPrefix = layerPrefix(layers[0].id)
       if(layers.length > 1) {
-        const groupPrefix = layerPrefix(layers[0].id)
         const grp = <LayerListGroup
-          key={'group-'+groupPrefix}
+          key={'group-' + groupPrefix + '-' + idx}
           title={groupPrefix}
-          isActive={true}
+          isActive={!isCollapsed(groupPrefix)}
+          onActiveToggle={this.toggleLayerGroup.bind(this, groupPrefix)}
         />
         listItems.push(grp)
       }
 
       layers.forEach(layer => {
         const listItem = <LayerListItem
+          className={classnames({
+            'maputnik-layer-list-item-collapsed': isCollapsed(groupPrefix)
+          })}
           index={idx}
           key={layer.id}
           layerId={layer.id}

src/components/layers/LayerListItem.jsx

@@ -61,6 +61,7 @@ class LayerListItem extends React.Component {
     layerType: React.PropTypes.string.isRequired,
     isSelected: React.PropTypes.bool,
     visibility: React.PropTypes.string,
+    className: React.PropTypes.string,
 
     onLayerSelect: React.PropTypes.func.isRequired,
     onLayerCopy: React.PropTypes.func,
@@ -93,6 +94,7 @@ class LayerListItem extends React.Component {
       className={classnames({
         "maputnik-layer-list-item": true,
         "maputnik-layer-list-item-selected": this.props.isSelected,
+        [this.props.className]: true,
       })}>
         <LayerTypeDragHandle type={this.props.layerType} />
         <span className="maputnik-layer-list-item-id">{this.props.layerId}</span>

src/styles/_layer.scss

@@ -62,6 +62,12 @@
     color: $color-white;
   }
 
+  &-item-collapsed {
+    height: 0;
+    overflow: hidden;
+    padding: 0;
+  }
+
   &-item-id {
     width: 115px;
     white-space: nowrap;