Use icon TYPES in prop type. Provide guidance on order of actions.

elastic · May 1, 2018 · 63bcbdf · 63bcbdf
1 parent 08f42b6
commit 63bcbdf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src-docs/src/views/empty_prompt/empty_prompt_example.js b/src-docs/src/views/empty_prompt/empty_prompt_example.js
@@ -72,7 +72,8 @@ export const EmptyPromptExample = {
           You can remove parts of the prompt to simplify it, if you wish.
         </p>
         <p>
-          You can also provide an array of multiple actions.
+          You can also provide an array of multiple actions. Be sure to list primary actions first
+          and secondary actions last.
         </p>
       </Fragment>
     ),

diff --git a/src/components/empty_prompt/empty_prompt.js b/src/components/empty_prompt/empty_prompt.js
@@ -5,7 +5,7 @@ import classNames from 'classnames';
 import { TITLE_SIZES } from '../title/title';
 import { EuiFlexGroup, EuiFlexItem } from '../flex';
 import { EuiSpacer } from '../spacer';
-import { EuiIcon, COLORS } from '../icon/icon';
+import { EuiIcon, COLORS, TYPES } from '../icon/icon';
 import { EuiText, EuiTextColor } from '../text';
 import { EuiTitle } from '../title';
 
@@ -114,7 +114,7 @@ export const EuiEmptyPrompt = ({
 };
 
 EuiEmptyPrompt.propTypes = {
-  iconType: PropTypes.string,
+  iconType: PropTypes.oneOf(TYPES),
 
   /**
    * Pass `null` to use original icon color