fix(office): Allow download of files via embedded links in Chrome (#1277

)
box · Nov 2, 2020 · 281b9ab · 281b9ab
1 parent 480a90d
commit 281b9ab
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/lib/viewers/office/OfficeViewer.js b/src/lib/viewers/office/OfficeViewer.js
@@ -268,7 +268,7 @@ class OfficeViewer extends BaseViewer {
         iframeEl.setAttribute('width', '100%');
         iframeEl.setAttribute('height', '100%');
         iframeEl.setAttribute('frameborder', 0);
-        iframeEl.setAttribute('sandbox', 'allow-scripts allow-same-origin allow-forms allow-popups');
+        iframeEl.setAttribute('sandbox', 'allow-scripts allow-same-origin allow-forms allow-popups allow-downloads');
 
         if (this.platformSetup) {
             iframeEl.setAttribute('allowfullscreen', 'true');

diff --git a/src/lib/viewers/office/__tests__/OfficeViewer-test.js b/src/lib/viewers/office/__tests__/OfficeViewer-test.js
@@ -268,7 +268,9 @@ describe('lib/viewers/office/OfficeViewer', () => {
             expect(iframeEl.width).toBe('100%');
             expect(iframeEl.height).toBe('100%');
             expect(iframeEl.frameBorder).toBe('0');
-            expect(iframeEl.getAttribute('sandbox')).toBe('allow-scripts allow-same-origin allow-forms allow-popups');
+            expect(iframeEl.getAttribute('sandbox')).toBe(
+                'allow-scripts allow-same-origin allow-forms allow-popups allow-downloads',
+            );
         });
 
         test('should allow fullscreen if using the platform setup', () => {