fix(reactstrap#2664): close offcanvas when fade is false (reactstrap#…

…2677)
humphreyja · Mar 2, 2023 · dac3587 · dac3587
1 parent fa1fcc8
commit dac3587
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/Offcanvas.js b/src/Offcanvas.js
@@ -425,6 +425,9 @@ class Offcanvas extends React.Component {
               classNames('offcanvas-backdrop', 'show', backdropClassName),
               cssModule,
             )}
+            ref={(c) => {
+              this._backdrop = c;
+            }}
             onClick={this.handleBackdropClick}
             onMouseDown={this.handleBackdropMouseDown}
           />

diff --git a/src/__tests__/Offcanvas.spec.js b/src/__tests__/Offcanvas.spec.js
@@ -379,6 +379,23 @@ describe('Offcanvas', () => {
     expect(toggle).toHaveBeenCalled();
   });
 
+  it('should call toggle when clicking backdrop when fade is false', () => {
+    const toggle = jest.fn();
+    render(
+      <Offcanvas isOpen toggle={toggle} fade={false}>
+        <button id="clicker">Does Nothing</button>
+      </Offcanvas>,
+    );
+
+    user.click(screen.getByText(/does nothing/i));
+
+    expect(toggle).not.toHaveBeenCalled();
+
+    user.click(document.getElementsByClassName('offcanvas-backdrop')[0]);
+
+    expect(toggle).toHaveBeenCalled();
+  });
+
   it('should destroy this._element', () => {
     const { rerender } = render(
       <Offcanvas isOpen toggle={toggle}>