From bdf2a661c803c5635a95393f9e2c32e2aa6d3e5e Mon Sep 17 00:00:00 2001 From: rajab Date: Sun, 25 Jun 2017 01:16:52 +0300 Subject: [PATCH] fix react-burger-menu definitions for pr --- types/react-burger-menu/index.d.ts | 72 ++++++++++--------- .../react-burger-menu-tests.tsx | 6 +- 2 files changed, 41 insertions(+), 37 deletions(-) diff --git a/types/react-burger-menu/index.d.ts b/types/react-burger-menu/index.d.ts index a05ce2215c..6d3a508c97 100644 --- a/types/react-burger-menu/index.d.ts +++ b/types/react-burger-menu/index.d.ts @@ -5,42 +5,46 @@ import * as React from "react"; -interface Props { - burgerBarClassName?: string; - burgerButtonClassName?: string; - crossButtonClassName?: string; - crossClassName?: string; - customBurgerIcon?: Element | false; - customCrossIcon?: Element | false; - id?: string; - isOpen?: boolean; - itemListClassName?: string; - menuClassName?: string; - morphShapeClassName?: string; - noOverlay?: boolean; - onStateChange?(): void; - // TODO (Rajab) This can be improved, though I do not know how. From PropTypes: - // styles && styles.outerContainer ? PropTypes.string.isRequired : PropTypes.string - outerContainerId?: string; - overlayClassName?: string; - // TODO (Rajab) This can be improved, though I do not know how. From PropTypes: - // styles && styles.pageWrap ? PropTypes.string.isRequired : PropTypes.string, - pageWrapId?: string; - right?: boolean; - styles?: { - bmBurgerButton: CSSStyleDeclaration; - bmBurgerBars: CSSStyleDeclaration; - bmCrossButton: CSSStyleDeclaration; - bmCross: CSSStyleDeclaration; - bmMenu: CSSStyleDeclaration; - bmMorphShape: CSSStyleDeclaration; - bmItemList: CSSStyleDeclaration; - bmOverlay: CSSStyleDeclaration; - }; - width?: number | string; +// TODO (Rajab) actually I don't want export Props, but +// `npm run lint react-burger-menu` error doesn't allow me to do it; +export interface Props { + burgerBarClassName?: string; + burgerButtonClassName?: string; + crossButtonClassName?: string; + crossClassName?: string; + customBurgerIcon?: Element | false; + customCrossIcon?: Element | false; + id?: string; + isOpen?: boolean; + itemListClassName?: string; + menuClassName?: string; + morphShapeClassName?: string; + noOverlay?: boolean; + onStateChange?(): void; + // TODO (Rajab) This can be improved, though I do not know how. From PropTypes: + // styles && styles.outerContainer ? PropTypes.string.isRequired : PropTypes.string + outerContainerId?: string; + overlayClassName?: string; + // TODO (Rajab) This can be improved, though I do not know how. From PropTypes: + // styles && styles.pageWrap ? PropTypes.string.isRequired : PropTypes.string, + pageWrapId?: string; + right?: boolean; + styles?: { + bmBurgerButton: CSSStyleDeclaration; + bmBurgerBars: CSSStyleDeclaration; + bmCrossButton: CSSStyleDeclaration; + bmCross: CSSStyleDeclaration; + bmMenu: CSSStyleDeclaration; + bmMorphShape: CSSStyleDeclaration; + bmItemList: CSSStyleDeclaration; + bmOverlay: CSSStyleDeclaration; + }; + width?: number | string; } -declare class ReactBurgerMenu extends React.Component { } +// TODO (Rajab) actually I don't want export ReactBurgerMenu, but +// `npm run lint react-burger-menu` error doesn't allow me to do it; +export class ReactBurgerMenu extends React.Component { } export class slide extends ReactBurgerMenu { } export class stack extends ReactBurgerMenu { } diff --git a/types/react-burger-menu/react-burger-menu-tests.tsx b/types/react-burger-menu/react-burger-menu-tests.tsx index 562b7ee3da..4a9190000b 100644 --- a/types/react-burger-menu/react-burger-menu-tests.tsx +++ b/types/react-burger-menu/react-burger-menu-tests.tsx @@ -1,9 +1,9 @@ import * as React from 'react'; import { slide as Menu } from 'react-burger-menu'; -class Example extends React.Component { +class Example extends React.Component<{}, {}> { showSettings(event: { - preventDefault: Function; + preventDefault(): void; }) { event.preventDefault(); } @@ -18,4 +18,4 @@ class Example extends React.Component { ); } -} \ No newline at end of file +}