Fix PDF view loading indicator (#12882)

RocketChat · Dec 10, 2018 · 70e6faa · 70e6faa
1 parent 490679a
commit 70e6faa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/packages/rocketchat-message-attachments/client/stylesheets/messageAttachments.css b/packages/rocketchat-message-attachments/client/stylesheets/messageAttachments.css
@@ -152,9 +152,11 @@ html.rtl .attachment {
 	& .attachment-pdf-loading {
 		display: none;
 
-		animation: spin 1s linear infinite;
-
 		font-size: 1.5rem;
+
+		svg {
+			animation: spin 1s linear infinite;
+		}
 	}
 
 	& .actions-container {

diff --git a/packages/rocketchat-ui-message/client/message.js b/packages/rocketchat-ui-message/client/message.js
@@ -33,7 +33,8 @@ async function renderPdfToCanvas(canvasId, pdfLink) {
 	const pdfjsLib = await import('pdfjs-dist');
 	pdfjsLib.GlobalWorkerOptions.workerSrc = `${ Meteor.absoluteUrl() }node_modules/pdfjs-dist/build/pdf.worker.js`;
 
-	const loader = document.getElementById('js-loading-${canvasId}');
+	const loader = document.getElementById(`js-loading-${ canvasId }`);
+
 	if (loader) {
 		loader.style.display = 'block';
 	}
@@ -45,10 +46,10 @@ async function renderPdfToCanvas(canvasId, pdfLink) {
 	const context = canvas.getContext('2d');
 	canvas.height = viewport.height;
 	canvas.width = viewport.width;
-	page.render({
+	await page.render({
 		canvasContext: context,
 		viewport,
-	});
+	}).promise;
 
 	if (loader) {
 		loader.style.display = 'none';