react-text-loop icon indicating copy to clipboard operation
react-text-loop copied to clipboard

Don't consider React.Fragments as node

Open ArthurBrito opened this issue 4 years ago • 0 comments

Firstly, thanks for the amazing lib!

I'm trying to achieve some like that: const ValueItems = () => ( <React.Fragment> ... </React.Fragment> ) <TextLoop interval={4500} adjustingSpeed={300}> <ValuesItems /> </TextLoop> As ValueItems is used in some places, it makes sense to me to create a component for that, however, to have the use ValueItems in TextLoop is not possible because it's wrapped with <React.Fragment>. The just check one level deep in the children nodes which I understand, however, I think that React.Fragments should be an exception to the rule as Fragments is intended to not be a node, but just a wrapper.

What are your thoughts?

ArthurBrito avatar Aug 13 '20 06:08 ArthurBrito