From caefc196a4ac44234d0823c5910cccb2aa840e2c Mon Sep 17 00:00:00 2001
From: Paulo Reis <paulojreis@gmail.com>
Date: Fri, 18 Nov 2016 14:27:49 +0000
Subject: [PATCH 1/2] Style comments in Overview panel.

---
 .../public/stylesheets/app/editor/review-panel.less    | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/services/web/public/stylesheets/app/editor/review-panel.less b/services/web/public/stylesheets/app/editor/review-panel.less
index e27cabfda3..8b56f4ac6c 100644
--- a/services/web/public/stylesheets/app/editor/review-panel.less
+++ b/services/web/public/stylesheets/app/editor/review-panel.less
@@ -203,6 +203,7 @@
 		}
 	}
 	.rp-state-overview & {
+		border-radius: 0;
 		padding: 2px 5px;
 		border-bottom: solid 1px @rp-border-grey;
 	}
@@ -328,6 +329,11 @@
 		display: flex;
 		align-items: flex-start;
 		padding: 5px;
+
+		.rp-state-overview & {
+			padding: 3px 0;
+			line-height: 1.2;
+		}
 	}
 		.rp-comment-body {
 			position: relative;
@@ -369,6 +375,10 @@
 
 		.rp-comment-reply {
 			padding: 0 5px;
+
+			.rp-state-overview & {
+				padding: 3px 0 0;
+			}
 		}
 
 	.rp-add-comment-btn {

From f029c7a7b2db2888fb09e8e0956bb53318eb470a Mon Sep 17 00:00:00 2001
From: Paulo Reis <paulojreis@gmail.com>
Date: Fri, 18 Nov 2016 15:08:16 +0000
Subject: [PATCH 2/2] Control click behaviour on comments; relayout when
 toggling panels.

---
 services/web/app/views/project/editor/review-panel.jade    | 1 +
 .../review-panel/controllers/ReviewPanelController.coffee  | 4 +---
 .../ide/review-panel/directives/reviewPanelSorted.coffee   | 7 +++++--
 .../web/public/stylesheets/app/editor/review-panel.less    | 1 +
 4 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/services/web/app/views/project/editor/review-panel.jade b/services/web/app/views/project/editor/review-panel.jade
index 7c3779b079..191108fa1d 100644
--- a/services/web/app/views/project/editor/review-panel.jade
+++ b/services/web/app/views/project/editor/review-panel.jade
@@ -154,6 +154,7 @@ script(type='text/ng-template', id='commentEntryTemplate')
 				textarea.rp-comment-input(
 					ng-model="entry.replyContent"
 					ng-keypress="handleCommentReplyKeyPress($event);"
+					stop-propagation="click"
 					placeholder="{{ 'Hit \"Enter\" to reply' + (entry.resolved ? ' and re-open' : '') }}"
 				)
 			.rp-comment-resolved-description(ng-if="entry.resolved && !entry.showWhenResolved")
diff --git a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee
index d6027d745c..63c9e6060c 100644
--- a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee
+++ b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee
@@ -38,8 +38,6 @@ define [
 		$scope.$watch "reviewPanel.subView", (subView) ->
 			return if !subView?
 			updateScrollbar()
-			if subView == $scope.SubViews.CUR_FILE
-				$scope.$broadcast "review-panel:layout"
 		
 		$scope.$watch "ui.reviewPanelOpen", (open) ->
 			return if !open?
@@ -155,7 +153,7 @@ define [
 
 		$scope.setSubView = (subView) -> 
 			$scope.reviewPanel.subView = subView
-		
+						
 		$scope.gotoEntry = (doc_id, entry) ->
 			console.log "Going to entry", entry.docPos
 			ide.editorManager.openDocId(doc_id, { gotoLine: entry.docPos.row + 1, gotoColumn: entry.docPos.column })
diff --git a/services/web/public/coffee/ide/review-panel/directives/reviewPanelSorted.coffee b/services/web/public/coffee/ide/review-panel/directives/reviewPanelSorted.coffee
index 1922c497f8..4faa233c0b 100644
--- a/services/web/public/coffee/ide/review-panel/directives/reviewPanelSorted.coffee
+++ b/services/web/public/coffee/ide/review-panel/directives/reviewPanelSorted.coffee
@@ -79,12 +79,15 @@ define [
 						entry.$callout_el.css(top: top + line_height + 1, height: original_top - top)
 						sl_console.log "ENTRY", {entry: entry.scope.entry, top}
 				
+				scope.$applyAsync () ->
+					layout()
+
 				scope.$watch "reviewPanel.entryGeneration", (value) ->
-					scope.$evalAsync () ->
+					scope.$applyAsync () ->
 						layout()
 				
 				scope.$on "review-panel:layout", () ->
-					scope.$evalAsync () ->
+					scope.$applyAsync () ->
 						layout()
 
 				## Scroll lock with Ace
diff --git a/services/web/public/stylesheets/app/editor/review-panel.less b/services/web/public/stylesheets/app/editor/review-panel.less
index 8b56f4ac6c..24589d416b 100644
--- a/services/web/public/stylesheets/app/editor/review-panel.less
+++ b/services/web/public/stylesheets/app/editor/review-panel.less
@@ -206,6 +206,7 @@
 		border-radius: 0;
 		padding: 2px 5px;
 		border-bottom: solid 1px @rp-border-grey;
+		cursor: pointer;
 	}
 
 	border-left: solid @rp-entry-ribbon-width transparent;