Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Commit 9090283

Browse files
committed
Pass a render prop to PullRequestReviewsContainer
Preserve existing review comments behavior in MultiFilePatchView.
1 parent 30bc7ce commit 9090283

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

lib/views/multi-file-patch-view.js

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@ import Commands, {Command} from '../atom/commands';
1616
import FilePatchHeaderView from './file-patch-header-view';
1717
import FilePatchMetaView from './file-patch-meta-view';
1818
import HunkHeaderView from './hunk-header-view';
19-
import PullRequestsReviewsContainer from '../containers/pr-reviews-container';
19+
import PullRequestReviewsContainer from '../containers/pr-reviews-container';
20+
import PullRequestReviewCommentThreadView from './pr-review-comment-thread-view';
2021
import RefHolder from '../models/ref-holder';
2122
import ChangedFileItem from '../items/changed-file-item';
2223
import CommitDetailItem from '../items/commit-detail-item';
@@ -385,13 +386,16 @@ export default class MultiFilePatchView extends React.Component {
385386
// "forceRerender" ensures that the PullRequestCommentsView re-renders each time that the MultiFilePatchView does.
386387
// It doesn't re-query for reviews, but it does re-check patch visibility.
387388
return (
388-
<PullRequestsReviewsContainer
389-
pullRequest={this.props.pullRequest}
390-
getBufferRowForDiffPosition={this.props.multiFilePatch.getBufferRowForDiffPosition}
391-
isPatchVisible={this.props.multiFilePatch.isPatchVisible}
392-
switchToIssueish={this.props.switchToIssueish}
393-
forceRerender={{}}
394-
/>
389+
<PullRequestReviewsContainer pullRequest={this.props.pullRequest} forceRerender={{}}>
390+
{commentThread => (
391+
<PullRequestReviewCommentThreadView
392+
commentThread={commentThread}
393+
getBufferRowForDiffPosition={this.props.multiFilePatch.getBufferRowForDiffPosition}
394+
isPatchVisible={this.props.multiFilePatch.isPatchVisible}
395+
switchToIssueish={this.props.switchToIssueish}
396+
/>
397+
)}
398+
</PullRequestReviewsContainer>
395399
);
396400
} else {
397401
return null;

0 commit comments

Comments
 (0)