Closes #1691; Fiz a grouping error in messages from history

RocketChat · Dec 17, 2015 · 4279690 · 4279690
1 parent 513034b
commit 4279690
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/packages/rocketchat-ui-message/message/message.coffee b/packages/rocketchat-ui-message/message/message.coffee
@@ -128,18 +128,29 @@ Template.message.onViewRendered = (context) ->
 
 			if previousDataset.date isnt currentDataset.date
 				$currentNode.addClass('new-day').removeClass('sequential')
+			else
+				$currentNode.removeClass('new-day')
 
-			if previousDataset.groupable is 'false' or previousDataset.username isnt currentDataset.username or parseInt(currentDataset.timestamp) - parseInt(previousDataset.timestamp) > RocketChat.settings.get('Message_GroupingPeriod') * 1000
+			if previousDataset.groupable is 'false'
 				$currentNode.removeClass('sequential')
+			else
+				if previousDataset.username isnt currentDataset.username or parseInt(currentDataset.timestamp) - parseInt(previousDataset.timestamp) > RocketChat.settings.get('Message_GroupingPeriod') * 1000
+					$currentNode.removeClass('sequential')
+				else
+					$currentNode.addClass('sequential')
 
 		if nextNode?.dataset?
 			nextDataset = nextNode.dataset
 
 			if nextDataset.date isnt currentDataset.date
 				$nextNode.addClass('new-day').removeClass('sequential')
+			else
+				$nextNode.removeClass('new-day')
 
 			if nextDataset.username isnt currentDataset.username or parseInt(nextDataset.timestamp) - parseInt(currentDataset.timestamp) > RocketChat.settings.get('Message_GroupingPeriod') * 1000
 				$nextNode.removeClass('sequential')
+			else
+				$nextNode.addClass('sequential')
 
 			if not nextNode?
 				templateInstance = view.parentView.parentView.parentView.parentView.parentView.templateInstance?()