Fix bailout broken in lazy components due to default props resolving by jddxf · Pull Request #18539 · react/react (original) (raw)

Conversation

@jddxf

Fix #17151

We should never compare unresolved props with resolved props. Since comparing resolved props by reference doesn't make sense, we use unresolved props in that case. Otherwise, resolved props are used.

@jddxf

@jddxf

We should never compare unresolved props with resolved props. Since comparing resolved props by reference doesn't make sense, we use unresolved props in that case. Otherwise, resolved props are used.

@jddxf

@codesandbox-ci

@jddxf jddxf mentioned this pull request

Apr 8, 2020

@sizebot

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 4e984ff

@sizebot

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 4e984ff

@gaearon

This was referenced

Mar 6, 2021

This was referenced

Mar 14, 2021

This was referenced

Mar 15, 2021

This was referenced

Sep 8, 2024

This was referenced

Sep 20, 2024

This was referenced

Oct 1, 2024

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Labels