diff --git a/services/web/app/views/project/editor/editor.pug b/services/web/app/views/project/editor/editor.pug index ce24108f77..6c062f3cbe 100644 --- a/services/web/app/views/project/editor/editor.pug +++ b/services/web/app/views/project/editor/editor.pug @@ -20,7 +20,6 @@ div.full-size( 'rp-size-expanded': ui.reviewPanelOpen,\ 'rp-layout-left': reviewPanel.layoutToLeft,\ 'rp-loading-threads': reviewPanel.loadingThreads,\ - 'rp-new-comment-ui': reviewPanel.newAddCommentUI\ }" ) .loading-panel(ng-show="!editor.sharejs_doc || editor.opening") diff --git a/services/web/app/views/project/editor/review-panel.pug b/services/web/app/views/project/editor/review-panel.pug index da14b69a99..2035af4775 100644 --- a/services/web/app/views/project/editor/review-panel.pug +++ b/services/web/app/views/project/editor/review-panel.pug @@ -8,7 +8,7 @@ ) !{translate("track_changes_is_on")} a.rp-add-comment-btn( href - ng-if="reviewPanel.newAddCommentUI && reviewPanel.entries[editor.open_doc_id]['add-comment'] != null" + ng-if="reviewPanel.entries[editor.open_doc_id]['add-comment'] != null" ng-click="addNewComment();" ) i.fa.fa-comment @@ -81,8 +81,6 @@ on-start-new="startNewComment();" on-submit="submitNewComment(content);" on-cancel="cancelNewComment();" - on-indicator-click="toggleReviewPanel();" - layout-to-left="reviewPanel.layoutToLeft" ) .rp-entry-list( @@ -322,14 +320,6 @@ script(type='text/ng-template', id='resolvedCommentEntryTemplate') script(type='text/ng-template', id='addCommentEntryTemplate') div .rp-entry-callout.rp-entry-callout-add-comment - .rp-entry-indicator.rp-entry-indicator-add-comment( - ng-if="!commentState.adding" - ng-click="startNewComment(); onIndicatorClick();" - tooltip=translate("add_comment") - tooltip-placement="{{ layoutToLeft ? 'left' : 'right' }}" - tooltip-append-to-body="true" - ) - i.fa.fa-commenting .rp-entry.rp-entry-add-comment( ng-class="[ (state.isAdding ? 'rp-entry-adding-comment' : ''), (entry.focused ? 'rp-entry-focused' : '')]" ) 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 fff1d31671..bf3f96ffc3 100644 --- a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee +++ b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee @@ -4,7 +4,7 @@ define [ "ide/colors/ColorManager" "ide/review-panel/RangesTracker" ], (App, EventEmitter, ColorManager, RangesTracker) -> - App.controller "ReviewPanelController", ($scope, $element, ide, $timeout, $http, $modal, event_tracking, sixpack, localStorage) -> + App.controller "ReviewPanelController", ($scope, $element, ide, $timeout, $http, $modal, event_tracking, localStorage) -> $reviewPanelEl = $element.find "#review-panel" $scope.SubViews = @@ -24,17 +24,8 @@ define [ loading: false commentThreads: {} resolvedThreadIds: {} - layoutToLeft: false rendererData: {} loadingThreads: false - newAddCommentUI: false # Test new UI for adding comments; remove afterwards. - - $scope.shouldABAddCommentBtn = false - if $scope.user.signUpDate >= '2017-03-27' - sixpack.participate "add-comment-btn", [ "default", "editor-corner" ], (variation) -> - $scope.shouldABAddCommentBtn = true - $scope.variationABAddCommentBtn = variation - $scope.reviewPanel.newAddCommentUI = (variation == "editor-corner") window.addEventListener "beforeunload", () -> collapsedStates = {} @@ -45,11 +36,9 @@ define [ localStorage("docs_collapsed_state:#{$scope.project_id}", valToStore) $scope.$on "layout:pdf:linked", (event, state) -> - $scope.reviewPanel.layoutToLeft = (state.east?.size < 220 || state.east?.initClosed) $scope.$broadcast "review-panel:layout" $scope.$on "layout:pdf:resize", (event, state) -> - $scope.reviewPanel.layoutToLeft = (state.east?.size < 220 || state.east?.initClosed) $scope.$broadcast "review-panel:layout", false $scope.$on "expandable-text-area:resize", (event) -> @@ -58,9 +47,6 @@ define [ $scope.$on "review-panel:sizes", (e, sizes) -> $scope.$broadcast "editor:set-scroll-size", sizes - - $scope.$watch "ui.pdfLayout", (layout) -> - $scope.reviewPanel.layoutToLeft = (layout == "flat") $scope.$watch "project.features.trackChangesVisible", (visible) -> return if !visible? @@ -173,7 +159,7 @@ define [ entries = $scope.reviewPanel.entries[$scope.editor.open_doc_id] or {} permEntries = {} for entry, entryData of entries - if entry != "add-comment" or !$scope.reviewPanel.newAddCommentUI + if entry != "add-comment" permEntries[entry] = entryData Object.keys(permEntries).length ), (nEntries) -> @@ -336,8 +322,6 @@ define [ $scope.$broadcast "comment:select_line" $timeout () -> $scope.$broadcast "review-panel:layout" - if $scope.shouldABAddCommentBtn and !$scope.ui.reviewPanelOpen - sixpack.convert "add-comment-btn" $scope.submitNewComment = (content) -> return if !content? or content == "" diff --git a/services/web/public/coffee/ide/review-panel/directives/addCommentEntry.coffee b/services/web/public/coffee/ide/review-panel/directives/addCommentEntry.coffee index 9d33bcb35a..a6cf315bfa 100644 --- a/services/web/public/coffee/ide/review-panel/directives/addCommentEntry.coffee +++ b/services/web/public/coffee/ide/review-panel/directives/addCommentEntry.coffee @@ -8,8 +8,6 @@ define [ onStartNew: "&" onSubmit: "&" onCancel: "&" - onIndicatorClick: "&" - layoutToLeft: "=" link: (scope, element, attrs) -> scope.state = isAdding: false diff --git a/services/web/public/stylesheets/app/editor/review-panel.less b/services/web/public/stylesheets/app/editor/review-panel.less index c069c536e2..59f0e40566 100644 --- a/services/web/public/stylesheets/app/editor/review-panel.less +++ b/services/web/public/stylesheets/app/editor/review-panel.less @@ -96,10 +96,12 @@ flex-direction: column; width: @review-panel-width; overflow: visible; + border-left-width: 1px; } .rp-size-mini & { width: @review-off-width; z-index: 6; + border-left-width: 1px; } position: absolute; @@ -107,7 +109,7 @@ bottom: 0px; right: 0px; background-color: @rp-bg-blue; - border-left: solid 1px @rp-border-grey; + border-left: solid 0px @rp-border-grey; font-size: @rp-base-font-size; color: @rp-type-blue; z-index: 6; @@ -181,6 +183,9 @@ .rp-size-mini & { display: block; } + .rp-size-mini &-add-comment { + display: none; + } position: absolute; left: 2px; right: 2px; @@ -200,9 +205,6 @@ z-index: 1; } - .rp-new-comment-ui &-add-comment { - display: none; - } } .rp-entry-wrapper { @@ -576,7 +578,7 @@ } } - .rp-size-mini.rp-new-comment-ui &-add-comment { + .rp-size-mini &-add-comment { display: none; } }