Skip to content

Commit 6be35c6

Browse files
committed
fix: CSidebar: emit onShowChange only on close
1 parent f6160fc commit 6be35c6

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/CSidebar.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ const CSidebar = props=>{
5858

5959
useEffect(() => {
6060
isOpen === true ? createBackdrop() : removeBackdrop()
61-
onShowChange && onShowChange(isOpen)
6261
return () => removeBackdrop()
6362
}, [isOpen])
6463

@@ -93,7 +92,10 @@ const CSidebar = props=>{
9392
}
9493
}
9594

96-
const closeSidebar = () => setIsOpen(overlaid ? false : 'responsive')
95+
const closeSidebar = () => {
96+
onShowChange && onShowChange(overlaid ? false : 'responsive')
97+
setIsOpen(overlaid ? false : 'responsive')
98+
}
9799

98100
const isOnMobile = ()=>{
99101
return Boolean(getComputedStyle(node.current).getPropertyValue('--is-mobile'))

0 commit comments

Comments
 (0)