fix: destroyOnClose should also works on inline mode (#281)

* test: test driven

* fix: inline should also support destroyOnClose

Author: zombieJ

src/Drawer.tsx

@@ -40,6 +40,10 @@ const Drawer: React.FC<DrawerProps> = props => {
     };
 
   // ============================ Render ============================
+  if (!forceRender && !animatedVisible && !open && destroyOnClose) {
+    return null;
+  }
+
   const sharedDrawerProps = {
     ...props,
     prefixCls,
@@ -50,10 +54,6 @@ const Drawer: React.FC<DrawerProps> = props => {
     return <DrawerPopup {...sharedDrawerProps} inline />;
   }
 
-  if (!forceRender && !animatedVisible && !open && destroyOnClose) {
-    return null;
-  }
-
   return (
     <Portal
       visible={open}

tests/index.spec.tsx

@@ -159,11 +159,23 @@ describe('rc-drawer-menu', () => {
     render(<Drawer forceRender />);
     expect(document.querySelector('.rc-drawer')).toBeTruthy();
   });
-  it('destroyOnClose', () => {
-    const { rerender } = render(<Drawer destroyOnClose open />);
-    expect(document.querySelector('.rc-drawer')).toBeTruthy();
-    rerender(<Drawer destroyOnClose />);
-    expect(document.querySelector('.rc-drawer')).toBeFalsy();
+
+  describe('destroyOnClose', () => {
+    it('basic', () => {
+      const { rerender } = render(<Drawer destroyOnClose open />);
+      expect(document.querySelector('.rc-drawer')).toBeTruthy();
+      rerender(<Drawer destroyOnClose />);
+      expect(document.querySelector('.rc-drawer')).toBeFalsy();
+    });
+
+    it('inline', () => {
+      const { container, rerender } = render(
+        <Drawer destroyOnClose open getContainer={false} />,
+      );
+      expect(container.querySelector('.rc-drawer')).toBeTruthy();
+      rerender(<Drawer destroyOnClose getContainer={false} />);
+      expect(container.querySelector('.rc-drawer')).toBeFalsy();
+    });
   });
 
   describe('placement', () => {