You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Since I realized that Reactstrap is already using classnames to handle className I started passing arrays directly the console.error below (hopefully soon it'll be a console.warn: facebook/prop-types#245)
I could of course call classnames my self, but it feels like it's not the best thing to do performance-wise (even though it's probably exactly what classnames will do with my array).
Of course, I understand that this would kind of bind the library to accepting class arrays, or at least bind it more...
Well, my point is, does anyone else think it'd be a good idea to allow also arrays and objects in the propTypes?
Error message in console
Failed prop type: Invalid prop `className` of type `array` supplied to `Col`, expected `string`
The text was updated successfully, but these errors were encountered:
What is happening?
Since I realized that Reactstrap is already using classnames to handle className I started passing arrays directly the console.error below (hopefully soon it'll be a console.warn: facebook/prop-types#245)
I could of course call classnames my self, but it feels like it's not the best thing to do performance-wise (even though it's probably exactly what classnames will do with my array).
Of course, I understand that this would kind of bind the library to accepting class arrays, or at least bind it more...
Well, my point is, does anyone else think it'd be a good idea to allow also arrays and objects in the propTypes?
Error message in console
The text was updated successfully, but these errors were encountered: