From ec9448bc188afb7aa3f0a77c7269517d5e37a2dd Mon Sep 17 00:00:00 2001
From: btzr-io <btzr.io@gmail.com>
Date: Wed, 4 Jul 2018 23:24:04 -0600
Subject: [PATCH] more fixes

unblock unplayable files

fix context menu errors of pdfViwer
---
 src/renderer/component/viewers/pdfViewer.jsx | 21 ++++++++++++++++----
 src/renderer/page/file/view.jsx              | 11 +++++++++-
 2 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/src/renderer/component/viewers/pdfViewer.jsx b/src/renderer/component/viewers/pdfViewer.jsx
index 80c4e94ab..26a136e0b 100644
--- a/src/renderer/component/viewers/pdfViewer.jsx
+++ b/src/renderer/component/viewers/pdfViewer.jsx
@@ -10,13 +10,26 @@ type Props = {
 };
 
 class PdfViewer extends React.PureComponent<Props> {
+  constructor(props) {
+    super(props);
+    this.viewer = React.createRef();
+  }
+
+  // TODO: Enable context-menu
+  stopContextMenu = event => {
+    event.preventDefault();
+    event.stopPropagation();
+  };
+
   render() {
     const { source } = this.props;
     return (
-      <webview
-        className="file-render__viewer"
-        src={`chrome://pdf-viewer/index.html?src=file://${source.downloadPath}`}
-      />
+      <div className="file-render__viewer" onContextMenu={this.stopContextMenu}>
+        <webview
+          ref={this.viewer}
+          src={`chrome://pdf-viewer/index.html?src=file://${source.downloadPath}`}
+        />
+      </div>
     );
   }
 }
diff --git a/src/renderer/page/file/view.jsx b/src/renderer/page/file/view.jsx
index 4d4e3f1b7..474b3a23f 100644
--- a/src/renderer/page/file/view.jsx
+++ b/src/renderer/page/file/view.jsx
@@ -50,7 +50,16 @@ type Props = {
 
 class FilePage extends React.Component<Props> {
   static PLAYABLE_MEDIA_TYPES = ['audio', 'video'];
-  static PREVIEW_MEDIA_TYPES = ['text', 'image', 'document', '3D-file'];
+  static PREVIEW_MEDIA_TYPES = [
+    'text',
+    'model',
+    'image',
+    '3D-file',
+    'document',
+    // Bypass unplayable files
+    // TODO: Find a better way to detect supported types
+    'application',
+  ];
 
   constructor(props: Props) {
     super(props);