Use more complete implementation of Omit in TypeScript definitions (#322

)

The previous implementation of the `Omit` type did not support optional and readonly fields, which led to type errors when using the components.

CHANGELOG.md

@@ -19,6 +19,7 @@
 **Bug fixes**
 
 - Set `EuiFlexGroup` to `flex-grow: 1` to be more friendly with IE11 [(#315)](https://github.com/elastic/eui/pull/315)
+- Fix TypeScript definitions such that optional and readonly properties survive being passed through `Omit` [(#322)](https://github.com/elastic/eui/pull/322)
 
 # [`0.0.13`](https://github.com/elastic/eui/tree/v0.0.13)
 

src/components/common.d.ts

@@ -18,5 +18,5 @@ declare module '@elastic/eui' {
   type Diff<T extends string, U extends string> = ({ [P in T]: P } &
     { [P in U]: never } & { [x: string]: never })[T];
 
-  type Omit<T, K extends keyof T> = { [P in Diff<keyof T, K>]: T[P] };
+  type Omit<T, K extends keyof T> = Pick<T, Diff<keyof T, K>>;
 }

src/components/context_menu/index.d.ts

@@ -90,13 +90,11 @@ declare module '@elastic/eui' {
     content?: React.ReactNode;
   }
 
-  export interface EuiContextMenuProps {
-    panels?: EuiContextMenuPanelDescriptor[];
-    initialPanelId?: EuiContextMenuPanelId;
-  }
+  export type EuiContextMenuProps = CommonProps &
+    Omit<HTMLAttributes<HTMLDivElement>, 'style'> & {
+      panels?: EuiContextMenuPanelDescriptor[];
+      initialPanelId?: EuiContextMenuPanelId;
+    };
 
-  export const EuiContextMenu: SFC<
-    Omit<HTMLAttributes<HTMLDivElement>, 'className' | 'style'> &
-      EuiContextMenuProps
-  >;
+  export const EuiContextMenu: SFC<EuiContextMenuProps>;
 }