Skip to content

Missing "children" prop feature in <Miss> #4026

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
stnwk opened this issue Oct 11, 2016 · 3 comments
Closed

Missing "children" prop feature in <Miss> #4026

stnwk opened this issue Oct 11, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@stnwk
Copy link

stnwk commented Oct 11, 2016

So, the <Match> Component takes a prop children which allows the following:

<Match children={({ matched, ...rest}) => (
  {/* Animate will always render, so you can use lifecycles
      to animate its children */}
  <Animate>
    {matched && (
      <Something {...rest}/>
    )}
  </Animate>
)}/>

As taken from the current v4 docs.

The function provided via this children prop will always be rendered, regardless whether it's a match or not - allowing to use lifecycle hooks in <Animated> to animate the content e.g. using ReactCSSTransitionGroup.

Unfortunately this prop is missing in the <Miss> component, making it unpossible to use lifecycles to animate it. Would be nice to add it.

@ryanflorence
Copy link
Member

thanks, yeah the rendering code was once shared, i'm going to share it
again with a new component so match and miss composition is easier for
others too
On Mon, Oct 10, 2016 at 6:32 PM Stefan [email protected] wrote:

So, the Component takes a prop children which allows the
following:

<Match children={({ matched, ...rest}) => (
{/* Animate will always render, so you can use lifecycles to animate its children */}

{matched && (
<Something {...rest}/>
)}

)}/>

As taken from the current v4 docs.

The function provided via this children prop will always be rendered,
regardless whether it's a match or not - allowing to use lifecycle hooks in
to animate the content e.g. using ReactCSSTransitionGroup.

Unfortunately this prop is missing in the component, making it
unpossible to use lifecycles to animate it. Would be nice to add it.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#4026, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAGHaJoGOZX4F4I_ZQpNW5JZiwD_wlsWks5qyucegaJpZM4KTJt4
.

@timdorr timdorr changed the title V4: Missing "children" prop feature in <Miss> Missing "children" prop feature in <Miss> Oct 11, 2016
@timdorr timdorr added this to the v4.0.0 milestone Oct 11, 2016
@timdorr timdorr added the bug label Oct 11, 2016
@ryanflorence ryanflorence self-assigned this Oct 26, 2016
@alisd23 alisd23 added the v4 label Nov 1, 2016
@timdorr timdorr removed the v4 label Nov 1, 2016
@supasate
Copy link
Contributor

supasate commented Nov 4, 2016

@ryanflorence I've made a PR to fix this issue (while the rendering code is not shared yet). I'm new here. Please advise if there's anything wrong.

@mjackson
Copy link
Member

<Miss> is gone in the current v4 branch. Closing.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

6 participants