[FIX] Enabling "Collapse Embedded Media by Default" hides replies, qu…

…otes (#10427) [FIX] Enabling `Collapse Embedded Media by Default` was hiding replies and quotes
RocketChat · May 18, 2018 · e358a3c · e358a3c
1 parent 89ff78a
commit e358a3c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/packages/rocketchat-message-attachments/client/messageAttachment.html b/packages/rocketchat-message-attachments/client/messageAttachment.html
@@ -62,7 +62,7 @@
 			{{/unless}}
 
 			{{#if image_url}}
-				{{#unless collapsed}}
+				{{#unless mediaCollapsed}}
 					<div class="attachment-image">
 					{{#if loadImage}}
 						<figure>
@@ -91,7 +91,7 @@
 			{{/if}}
 
 			{{#if audio_url}}
-				{{#unless collapsed}}
+				{{#unless mediaCollapsed}}
 					<div class="attachment-audio">
 						<audio controls>
 							<source src="{{fixCordova audio_url}}" type="{{audio_type}}" data-description="{{description}}">
@@ -102,7 +102,7 @@
 			{{/if}}
 
 			{{#if video_url}}
-				{{#unless collapsed}}
+				{{#unless mediaCollapsed}}
 					<div class="attachment-video">
 						<video controls class="inline-video">
 							<source src="{{fixCordova video_url}}" type="{{video_type}}" data-description="{{description}}">

diff --git a/packages/rocketchat-message-attachments/client/messageAttachment.js b/packages/rocketchat-message-attachments/client/messageAttachment.js
@@ -52,6 +52,12 @@ Template.messageAttachment.helpers({
 		return colors[this.color] || this.color;
 	},
 	collapsed() {
+		if (this.collapsed != null) {
+			return this.collapsed;
+		}
+		return false;
+	},
+	mediaCollapsed() {
 		if (this.collapsed != null) {
 			return this.collapsed;
 		} else {