From a91a1272d1421ec2bce37bcd3ba243de5204f5f7 Mon Sep 17 00:00:00 2001
From: Krist Wongsuphasawat <krist.wongz@gmail.com>
Date: Mon, 27 Apr 2020 13:37:12 -0700
Subject: [PATCH] fix: address comments

---
 .../stories/plugins/plugin-chart-word-cloud/Stories.tsx       | 4 ++--
 .../stories/plugins/preset-chart-xy/Line/Stories.tsx          | 2 ++
 .../stories/plugins/preset-chart-xy/ScatterPlot/Stories.tsx   | 2 ++
 .../storybook/stories/superset-ui-chart/SuperChartStories.tsx | 2 +-
 .../stories/superset-ui-color/ColorPallettesStories.jsx       | 2 +-
 .../stories/superset-ui-connection/ConnectionStories.tsx      | 2 +-
 .../stories/superset-ui-number-format/BigNumberStories.jsx    | 2 +-
 .../stories/superset-ui-time-format/TimeFormatStories.jsx     | 2 +-
 8 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/packages/superset-ui-demo/storybook/stories/plugins/plugin-chart-word-cloud/Stories.tsx b/packages/superset-ui-demo/storybook/stories/plugins/plugin-chart-word-cloud/Stories.tsx
index 36826751c8..c6b4a1492c 100644
--- a/packages/superset-ui-demo/storybook/stories/plugins/plugin-chart-word-cloud/Stories.tsx
+++ b/packages/superset-ui-demo/storybook/stories/plugins/plugin-chart-word-cloud/Stories.tsx
@@ -1,6 +1,6 @@
 import React from 'react';
 import { SuperChart } from '@superset-ui/chart';
-import { select } from '@storybook/addon-knobs';
+import { select, withKnobs } from '@storybook/addon-knobs';
 import WordCloudChartPlugin from '@superset-ui/plugin-chart-word-cloud';
 import LegacyWordCloudChartPlugin from '@superset-ui/plugin-chart-word-cloud/esm/legacy';
 import data from './data';
@@ -10,6 +10,7 @@ new LegacyWordCloudChartPlugin().configure({ key: 'legacy-word-cloud2' }).regist
 
 export default {
   title: 'Chart Plugins|plugin-chart-word-cloud',
+  decorators: [withKnobs],
 };
 
 export const basic = () => (
@@ -78,7 +79,6 @@ export const encodesColorByWordLength = () => (
   />
 );
 
-
 export const encodesFontByFirstLetter = () => (
   <SuperChart
     chartType="word-cloud2"
diff --git a/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/Line/Stories.tsx b/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/Line/Stories.tsx
index e27f9e043c..524e8e1430 100644
--- a/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/Line/Stories.tsx
+++ b/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/Line/Stories.tsx
@@ -1,11 +1,13 @@
 import { LineChartPlugin, LegacyLineChartPlugin } from '@superset-ui/preset-chart-xy';
 import { LINE_PLUGIN_TYPE, LINE_PLUGIN_LEGACY_TYPE } from './constants';
+import { withKnobs } from '@storybook/addon-knobs';
 
 new LegacyLineChartPlugin().configure({ key: LINE_PLUGIN_LEGACY_TYPE }).register();
 new LineChartPlugin().configure({ key: LINE_PLUGIN_TYPE }).register();
 
 export default {
   title: 'Chart Plugins|preset-chart-xy/Line',
+  decorators: [withKnobs],
 };
 
 export { default as basic } from './stories/basic';
diff --git a/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/ScatterPlot/Stories.tsx b/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/ScatterPlot/Stories.tsx
index 0823effb8e..a404cbf26b 100644
--- a/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/ScatterPlot/Stories.tsx
+++ b/packages/superset-ui-demo/storybook/stories/plugins/preset-chart-xy/ScatterPlot/Stories.tsx
@@ -1,11 +1,13 @@
 import { ScatterPlotChartPlugin, LegacyScatterPlotChartPlugin } from '@superset-ui/preset-chart-xy';
 import { SCATTER_PLOT_PLUGIN_TYPE, SCATTER_PLOT_PLUGIN_LEGACY_TYPE } from './constants';
+import { withKnobs } from '@storybook/addon-knobs';
 
 new LegacyScatterPlotChartPlugin().configure({ key: SCATTER_PLOT_PLUGIN_LEGACY_TYPE }).register();
 new ScatterPlotChartPlugin().configure({ key: SCATTER_PLOT_PLUGIN_TYPE }).register();
 
 export default {
   title: 'Chart Plugins|preset-chart-xy/ScatterPlot',
+  decorators: [withKnobs],
 };
 
 export { default as basic } from './stories/basic';
diff --git a/packages/superset-ui-demo/storybook/stories/superset-ui-chart/SuperChartStories.tsx b/packages/superset-ui-demo/storybook/stories/superset-ui-chart/SuperChartStories.tsx
index 4977f60c81..4f86bb19ac 100644
--- a/packages/superset-ui-demo/storybook/stories/superset-ui-chart/SuperChartStories.tsx
+++ b/packages/superset-ui-demo/storybook/stories/superset-ui-chart/SuperChartStories.tsx
@@ -13,7 +13,7 @@ new BuggyChartPlugin().configure({ key: ChartKeys.BUGGY }).register();
 const DEFAULT_QUERY_DATA = { data: ['foo', 'bar'] };
 
 export default {
-  title: 'Core Packages|@superset-ui/chart',
+  title: '*Core Packages|@superset-ui/chart',
   decorators: [withKnobs],
 };
 
diff --git a/packages/superset-ui-demo/storybook/stories/superset-ui-color/ColorPallettesStories.jsx b/packages/superset-ui-demo/storybook/stories/superset-ui-color/ColorPallettesStories.jsx
index 4874dffd55..08fe7a2805 100644
--- a/packages/superset-ui-demo/storybook/stories/superset-ui-color/ColorPallettesStories.jsx
+++ b/packages/superset-ui-demo/storybook/stories/superset-ui-color/ColorPallettesStories.jsx
@@ -11,7 +11,7 @@ import SequantialD3Palettes from '@superset-ui/color/src/colorSchemes/sequential
 import RenderPalettes from './RenderPalettes';
 
 export default {
-  title: 'Core Packages|@superset-ui/color',
+  title: '*Core Packages|@superset-ui/color',
 };
 
 export const categoricalPalettes = () =>
diff --git a/packages/superset-ui-demo/storybook/stories/superset-ui-connection/ConnectionStories.tsx b/packages/superset-ui-demo/storybook/stories/superset-ui-connection/ConnectionStories.tsx
index df838532e2..a6ff795899 100644
--- a/packages/superset-ui-demo/storybook/stories/superset-ui-connection/ConnectionStories.tsx
+++ b/packages/superset-ui-demo/storybook/stories/superset-ui-connection/ConnectionStories.tsx
@@ -8,7 +8,7 @@ import Expandable from '../../shared/components/Expandable';
 const REQUEST_METHODS = ['GET', 'POST'];
 
 export default {
-  title: 'Core Packages|@superset-ui/connection',
+  title: '*Core Packages|@superset-ui/connection',
   decorators: [withKnobs],
 };
 
diff --git a/packages/superset-ui-demo/storybook/stories/superset-ui-number-format/BigNumberStories.jsx b/packages/superset-ui-demo/storybook/stories/superset-ui-number-format/BigNumberStories.jsx
index 87fc43e570..f2a1b12177 100644
--- a/packages/superset-ui-demo/storybook/stories/superset-ui-number-format/BigNumberStories.jsx
+++ b/packages/superset-ui-demo/storybook/stories/superset-ui-number-format/BigNumberStories.jsx
@@ -116,7 +116,7 @@ NumberFormatValidator.propTypes = propTypes;
 NumberFormatValidator.defaultProps = defaultProps;
 
 export default {
-  title: 'Core Packages|@superset-ui/number-format',
+  title: '*Core Packages|@superset-ui/number-format',
 };
 
 export const validator = () => <NumberFormatValidator />;
diff --git a/packages/superset-ui-demo/storybook/stories/superset-ui-time-format/TimeFormatStories.jsx b/packages/superset-ui-demo/storybook/stories/superset-ui-time-format/TimeFormatStories.jsx
index da71833571..6ccdc96468 100644
--- a/packages/superset-ui-demo/storybook/stories/superset-ui-time-format/TimeFormatStories.jsx
+++ b/packages/superset-ui-demo/storybook/stories/superset-ui-time-format/TimeFormatStories.jsx
@@ -105,7 +105,7 @@ TimeFormatValidator.propTypes = propTypes;
 TimeFormatValidator.defaultProps = defaultProps;
 
 export default {
-  title: 'Core Packages|@superset-ui/time-format',
+  title: '*Core Packages|@superset-ui/time-format',
 };
 
 export const validator = () => <TimeFormatValidator />;