Skip to content

Commit 6ab8895

Browse files
committed
fix(CCarousel): put setState into useEffect
1 parent f7e57dc commit 6ab8895

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/components/carousel/CCarouselInner.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { Children, forwardRef, HTMLAttributes, useContext } from 'react'
1+
import React, { Children, forwardRef, HTMLAttributes, useContext, useEffect } from 'react'
22
import PropTypes from 'prop-types'
33
import classNames from 'classnames'
44

@@ -16,7 +16,9 @@ export const CCarouselInner = forwardRef<HTMLDivElement, CCarouselInnerProps>(
1616
const { setItemsNumber } = useContext(Context)
1717
const _className = classNames('carousel-inner', className)
1818

19-
setItemsNumber(Children.toArray(children).length)
19+
useEffect(() => {
20+
setItemsNumber(Children.toArray(children).length)
21+
})
2022

2123
return (
2224
<div className={_className} {...rest} ref={ref}>

0 commit comments

Comments
 (0)