From 9d2c9923c45cef9fd5dbd3b66ef1899b059fc34f Mon Sep 17 00:00:00 2001
From: Michael An <37589122+Michael18811380328@users.noreply.github.com>
Date: Mon, 18 Feb 2019 14:07:38 +0800
Subject: [PATCH] handle file change equal to zero (#2954)

---
 frontend/src/css/draft-review.css | 7 +++++--
 frontend/src/draft-review.js      | 4 ++--
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/frontend/src/css/draft-review.css b/frontend/src/css/draft-review.css
index c6dc938906..85bbc08930 100644
--- a/frontend/src/css/draft-review.css
+++ b/frontend/src/css/draft-review.css
@@ -88,7 +88,9 @@
   font-weight: 700;
 }
 .review-side-panel-body {
-  padding: 1rem 2rem;
+  padding: 1rem 2rem 3rem;
+  overflow-y: scroll;
+  height: 100%;
 }
 .review-side-panel-body .dirent-table-container {
   padding-left: 0;
@@ -190,7 +192,8 @@
 .review-side-panel .tab-content {
   height: 100%;
 }
-.review-side-panel .tab-content .comments {
+.review-side-panel .tab-content .comments,
+.review-side-panel .tab-content .tab-pane {
   height: 100%;
 }
 
diff --git a/frontend/src/draft-review.js b/frontend/src/draft-review.js
index 80e7464c04..f9d8f0eb32 100644
--- a/frontend/src/draft-review.js
+++ b/frontend/src/draft-review.js
@@ -884,10 +884,10 @@ class DraftReview extends React.Component {
                     <div className="review-side-panel-body">
                       <SidePanelReviewers
                         reviewers={this.state.reviewers}
-                        toggleAddReviewerDialog={this.toggleAddReviewerDialog}/>                        
+                        toggleAddReviewerDialog={this.toggleAddReviewerDialog}/>
                       <SidePanelAuthor/>
                       <UnresolvedComments number={this.state.unresolvedComments}/>
-                      { this.state.isShowDiff &&
+                      {(this.state.isShowDiff === true && this.state.changedNodes.length > 0) &&
                       <SidePanelChanges
                         changedNumber={this.state.changedNodes.length}
                         scrollToChangedNode={this.scrollToChangedNode}/>