Browse Source

Track timestamps of changes

James Allen 10 years ago
parent
commit
446ca7212a

+ 1 - 0
services/web/app/views/project/editor/editor.jade

@@ -51,6 +51,7 @@ div.full-size(
 			.review-panel-scroller
 				.review-entry-list(review-panel-sorted)
 					.review-entry(ng-repeat="(entry_id, entry) in reviewPanel.entries", ng-style="{'top': top}")
+						div.small {{ entry.metadata.ts }}
 						{{ entry.content }}
 
 	.ui-layout-east

+ 4 - 0
services/web/public/coffee/ide/editor/directives/aceEditor/track-changes/ChangesTracker.coffee

@@ -59,8 +59,10 @@ define [
 				offset, length, metadata
 			}
 			@emit "comment:added", comment
+			return comment
 		
 		applyOp: (op, metadata) ->
+			metadata.ts ?= new Date()
 			# Apply an op that has been applied to the document to our changes to keep them up to date
 			if op.i?
 				@applyInsertToChanges(op, metadata)
@@ -149,6 +151,7 @@ define [
 							is_same_user
 						offset = op_start - change_start
 						change.op.i = change.op.i.slice(0, offset) + op.i + change.op.i.slice(offset)
+						change.metadata.ts = metadata.ts
 						already_merged = true
 						moved_changes.push change
 					else if op_start <= change_start
@@ -241,6 +244,7 @@ define [
 						if insert_remaining.length > 0
 							change.op.i = insert_remaining
 							change.op.p = Math.min(change_start, op_start)
+							change.metadata.ts = metadata.ts
 							moved_changes.push change
 						else
 							remove_changes.push change

+ 1 - 0
services/web/public/coffee/ide/editor/directives/aceEditor/track-changes/TrackChangesManager.coffee

@@ -143,6 +143,7 @@ define [
 				@$scope.reviewPanel.entries[change.id] = {
 					content: change.op.i or change.op.d
 					offset: change.op.p
+					metadata: change.metadata
 				}
 			for comment in @changesTracker.comments
 				@$scope.reviewPanel.entries[comment.id] = {