-
Notifications
You must be signed in to change notification settings - Fork 14k
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
chore: migrate AnchorLink component from jsx to tsx #17647
chore: migrate AnchorLink component from jsx to tsx #17647
Conversation
@etr2460 Ptal, thx! |
if (inFocus) { | ||
this.scrollToView(); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resources referred to replace the UNSAFE_componentWillReceiveProps
:
if (inFocus) { | ||
scrollToView(); | ||
} | ||
}, [inFocus]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replaces UNSAFE_componentWillReceiveProps
method
dce84e3
to
97bb8e4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
once this is building, let's bring up a testenv so we can validate that this all works as expected
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue |
SUMMARY
PR for migrating
AnchorLink
react component from JSX to TSXBEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION