Update the ClayDatasetDisplay use of ClayManagementToolbar | Fixes li…

…feray#543
matuzalemsteles · Feb 16, 2018 · 2addee5 · 2addee5
1 parent 078b112
commit 2addee5
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 10 deletions.
diff --git a/packages/clay-dataset-display/src/ClayDatasetDisplay.js b/packages/clay-dataset-display/src/ClayDatasetDisplay.js
@@ -273,15 +273,19 @@ ClayDatasetDisplay.STATE = {
 	 * Configuration of the management bar plus button.
 	 * @instance
 	 * @memberof ClayDatasetDisplay
-	 * @type {?object|undefined}
+	 * @type {?object|string|bool|undefined}
 	 * @default undefined
 	 */
-	creationMenu: Config.shapeOf({
-		button: Config.object(),
-		caption: Config.string(),
-		helpText: Config.string(),
-		items: actionItemsValidator,
-	}),
+	creationMenu: Config.oneOfType([
+		Config.bool().value(false),
+		Config.string(),
+		Config.shapeOf({
+			button: Config.object(),
+			caption: Config.string(),
+			helpText: Config.string(),
+			items: actionItemsValidator,
+		}),
+	]),
 
 	/**
 	 * CSS classes to be applied to the element.

diff --git a/packages/clay-dataset-display/src/ClayDatasetDisplay.soy b/packages/clay-dataset-display/src/ClayDatasetDisplay.soy
@@ -14,7 +14,7 @@
 		separator: bool,
 		type: string
 	]>}
-	{@param? creationMenu: [
+	{@param? creationMenu: bool|string|[
 		button: ?,
 		caption: string,
 		helpText: string,
@@ -212,4 +212,4 @@
 			{/call}
 		</form>
 	</div>
-{/deltemplate}
+{/deltemplate}
diff --git a/packages/clay-dataset-display/src/__tests__/ClayDatasetDisplay.js b/packages/clay-dataset-display/src/__tests__/ClayDatasetDisplay.js
@@ -527,7 +527,7 @@ describe('ClayDatasetDisplay', function() {
 	it('should render a ClayDatasetDisplay and emit an event on management toolbar plus button click', () => {
 		jest.useFakeTimers();
 
-		defaultConfig.creationMenu = undefined;
+		defaultConfig.creationMenu = true;
 
 		component = new ClayDatasetDisplay(defaultConfig);