-
Notifications
You must be signed in to change notification settings - Fork 162
Unknown prop warnings - React 15.2.0 #114
Comments
I'm having the same error here with React 15.3.0. |
Thanks for calling this out, working on a PR to fix this now! |
@tonybaroneee FF to send to me. I've been on vacation but I need to get back to maintaining this project. |
Sure thing, good to know. Hope you're enjoying yourself! |
@Moeface @luisherranz Can you clarify which component you were seeing this on? |
This was in VelocityComponent |
@finneganh Any thoughts on this? I don't see any property spreading happening in |
This is the warning I am getting:
Curious thing is that I have updated the project to React 15.3.0 and that should fix the ReactTransitionGroup span problem, but it doesn't. Is VelocityTransitionGroup using its own ReactTransitionGroup or does it get it from React (peer dependency)? |
It's using |
This is the code on my
Shouldn't that use the react version of my project, which is in fact 15.3.0, which should solve the span warning?
|
Ah right, I don't believe it matters since the actual logic is just embedded within React. |
Maybe this is a bug in |
Ah, so neither As for |
Shit, you are right, that was my fault. I'm sorry. It looks like while I was searching I read this issue:
which was a real bug and I assumed this was a bug as well. Anyway, thanks for the help! |
No worries, glad you have it figured out! |
Since React 15.2.0, warnings have been added when unknown/invalid props are passed to regular DOM elements. React-bootstrap and many other libraries have had to modify the way props are passed in their modules. Relevant discussion is contained in this issue: react-bootstrap/react-bootstrap#1994
velocity-react is having the same issues, as I'm receiving the following warnings:
Warning: Unknown prop 'duration' on <span> tag. Remove this prop from the element.
Thanks for the great module, it's been working a treat otherwise!
The text was updated successfully, but these errors were encountered: