feat(Transition): Only call findDOMNode if necessary

[eps1lon]

Another approach to #457. It only calls findDOMNode if any of the callbacks has the node argument specified.

This is only exploratory since seemingly unrelated implementation had to be change. Details at the affected line.

[eps1lon]

Has its own commit (87832e9). No test broke upon removing this so I need some guidance to understand this condition.

[dimensi]

#464

[eps1lon]

That PR did not introduce the evaluation of node.

[jquense]

this is cool, the problem nowdays with this approach tho is destructuring messings with the observable arity of the function e.g.

function foo(...args) {
  console.log(args
}
foo.length // 0

[eps1lon]

this is cool, the problem nowdays with this approach tho is destructuring messings with the observable arity of the function e.g.

function foo(...args) {
  console.log(args
}
foo.length // 0

It was to good to be true. This would be a breaking change then at which point we might as well remove the call completely.
Technical explanation that I missed:

This number excludes the rest parameter and only includes parameters before the first one with a default value.
-- https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/length#Description

Any thoughts about #468 (comment)?

[eps1lon]

Fixed with #559