Fix new type definitions, clean up some props

elastic · Jan 22, 2018 · c593b30 · c593b30
1 parent 2217976
commit c593b30
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/src/components/form/radio/index.d.ts b/src/components/form/radio/index.d.ts
@@ -17,7 +17,7 @@ declare module '@elastic/eui' {
     Omit<HTMLAttributes<HTMLDivElement>, 'onChange'> & {
       options?: EuiRadioGroupOption[];
       idSelected?: string;
-      onChange: (id: string) => void;
+      onChange: EuiRadioGroupChangeCallback;
     };
 
   export type x = EuiRadioGroupProps['onChange'];

diff --git a/src/components/loading/loading_chart.js b/src/components/loading/loading_chart.js
@@ -32,6 +32,10 @@ export const EuiLoadingChart = ({ size, mono, className, ...rest }) => {
 };
 
 EuiLoadingChart.propTypes = {
-  size: PropTypes.oneOf(SIZES),
+  mono: PropTypes.bool,
+  size: PropTypes.oneOf(SIZES)
 };
 
+EuiLoadingChart.defaultProps = {
+  mono: false
+};
diff --git a/src/components/progress/index.d.ts b/src/components/progress/index.d.ts
@@ -10,7 +10,7 @@ declare module '@elastic/eui' {
     | 'accent'
     | 'danger'
     | 'primary'
-    | 'secondar'
+    | 'secondary'
     | 'subdued';
 
   export type EuiProgressSize = 'xs' | 's' | 'm' | 'l';

diff --git a/src/components/progress/progress.js b/src/components/progress/progress.js
@@ -74,7 +74,6 @@ EuiProgress.propTypes = {
   color: PropTypes.oneOf(COLORS),
   position: PropTypes.oneOf(POSITIONS),
   max: PropTypes.number,
-  indeterminate: PropTypes.bool,
 };
 
 EuiProgress.defaultProps = {