Update: treepicker simple pure option to hide the svg icon

Adslot · Dec 7, 2016 · 2cee378 · 2cee378
1 parent 5d6d7e7
commit 2cee378
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 1 deletion.
diff --git a/src/components/Main.jsx b/src/components/Main.jsx
@@ -455,9 +455,11 @@ class AppComponent extends React.Component {
 
         <TreePickerSimplePure
           itemType={this.state.itemType}
+          hideIcon
           selectedNodes={[]}
           subtree={this.state.treePickerPureSubtree}
-          initialStateNode={<div><h><b>Start by searching for items</b></h></div>}
+          emptySelectedListText={<div><b>Choose items of interest</b></div>}
+          initialStateNode={<div><b>Start by searching for items</b></div>}
           searchValue={this.state.searchValue}
           searchOnChange={this.setSearchValue}
           searchOnClear={this.searchOnClear}

diff --git a/src/components/adslotUi/TreePickerGridComponent.jsx b/src/components/adslotUi/TreePickerGridComponent.jsx
@@ -11,6 +11,7 @@ const TreePickerGridComponent = ({
   disabled,
   emptySvgSymbol,
   expandNode,
+  hideIcon,
   includeNode,
   itemType,
   nodes,
@@ -40,6 +41,7 @@ const TreePickerGridComponent = ({
     {nodes ?
       <Empty
         collection={nodes}
+        hideIcon={hideIcon}
         svgSymbol={emptySvgSymbol}
         text={emptyText}
       /> :
@@ -54,6 +56,7 @@ TreePickerGridComponent.propTypes = {
   emptySvgSymbol: PropTypes.shape(SvgSymbol.propTypes),
   emptyText: PropTypes.any.isRequired,
   expandNode: PropTypes.func,
+  hideIcon: PropTypes.bool,
   includeNode: PropTypes.func,
   itemType: PropTypes.string.isRequired,
   nodes: PropTypes.arrayOf(TreePickerPropTypes.node),
@@ -65,6 +68,7 @@ TreePickerGridComponent.propTypes = {
 
 TreePickerGridComponent.defaultProps = {
   disabled: false,
+  hideIcon: false,
 };
 
 export default TreePickerGridComponent;
diff --git a/src/components/adslotUi/TreePickerSimplePureComponent.jsx b/src/components/adslotUi/TreePickerSimplePureComponent.jsx
@@ -20,6 +20,7 @@ const TreePickerSimplePureComponent = ({
   emptyText,
   emptySelectedListText,
   expandNode,
+  hideIcon,
   includeNode,
   initialStateNode,
   initialStateSymbol,
@@ -82,6 +83,7 @@ const TreePickerSimplePureComponent = ({
             disabled,
             emptySvgSymbol: emptySelectedListSvgSymbol || emptySvgSymbol,
             emptyText: emptySelectedListText || 'Nothing selected.',
+            hideIcon,
             itemType,
             nodes: selectedNodes,
             nodeRenderer,
@@ -108,6 +110,7 @@ TreePickerSimplePureComponent.propTypes = {
   emptyText: PropTypes.any,
   emptySelectedListText: PropTypes.any,
   expandNode: PropTypes.func,
+  hideIcon: PropTypes.bool,
   includeNode: PropTypes.func,
   initialStateNode: PropTypes.any,
   initialStateSymbol: PropTypes.shape(SvgSymbol.propTypes),

diff --git a/test/components/adslotUi/TreePickerSimplePureComponentTest.jsx b/test/components/adslotUi/TreePickerSimplePureComponentTest.jsx
@@ -108,4 +108,14 @@ describe('TreePickerSimplePureComponent', () => {
     expect(component.find(TreePickerGrid).first().prop('No items to select.'));
     expect(component.find(TreePickerGrid).last().prop('Nothing to show'));
   });
+
+  it('should render with given empty selected state node', () => {
+    const loaderStateProps = _.assign({}, props, {
+      initialSelectedStateNode: 'Select...',
+      selectedNodes: [],
+    });
+
+    const component = shallow(<TreePickerSimplePure {...loaderStateProps} />);
+    expect(component.find(TreePickerGrid).last().prop('Select...'));
+  });
 });