Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Apr 20, 2016

#116 This worked for me. I'm no longer experiencing re-renders caused bu actions.

This worked for me. I'm no longer experiencing re-renders caused bu actions.
// This could leads to issues like below:
// https://github.com/kadirahq/react-storybook/issues/81
ReactDOM.unmountComponentAtNode(rootEl);
if (selectedKind !== previousKind || previousStory !== selectedStory) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a comment on why we do this and add the link to the related issue.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment added

@arunoda arunoda merged commit ef3db04 into storybookjs:master Apr 20, 2016
@arunoda
Copy link
Member

arunoda commented Apr 20, 2016

Thanks.

@arunoda
Copy link
Member

arunoda commented Apr 20, 2016

Released with v1.16.0

wyattdanger pushed a commit to wyattdanger/react-storybook that referenced this pull request Apr 26, 2016
* fix for re-render issue

This worked for me. I'm no longer experiencing re-renders caused bu actions.

* changed const to let

* added comment and reference to issue

* fixed typo

* code style
ndelangen pushed a commit that referenced this pull request Apr 5, 2017
Updated react-native in peerDependencies
@shilman shilman added the misc label May 27, 2017
@ghost ghost mentioned this pull request Nov 24, 2017
@nx-cloud
Copy link

nx-cloud bot commented Apr 8, 2024

View your CI Pipeline Execution ↗ for commit f1510b4

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 1m 16s View ↗

☁️ Nx Cloud last updated this comment at 2025-09-18 02:55:25 UTC

@coderabbitai coderabbitai bot mentioned this pull request Sep 24, 2025
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants