[MS] Use Cryptpad instead of viewers when possible

Author: Max-7

client/src/common/fileTypes.ts

@@ -11,6 +11,7 @@ enum FileContentType {
   Document = 'document',
   Text = 'text',
   PdfDocument = 'pdf-document',
+  Presentation = 'presentation',
   Unknown = 'unknown',
 }
 
@@ -25,6 +26,7 @@ const DOCUMENTS = ['docx'];
 const PDF_DOCUMENTS = ['pdf'];
 const AUDIOS = ['wav', 'mp3', 'ogg'];
 const VIDEOS = ['mp4', 'mpeg', 'webm'];
+const PRESENTATIONS = ['pptx'];
 
 // For generic text/plain
 const TEXTS = [
@@ -80,7 +82,7 @@ async function getMimeTypeFromBuffer(data: Uint8Array): Promise<string | undefin
   }
 }
 
-function detectFileContentType(name: EntryName): DetectedFileType | undefined {
+function detectFileContentType(name: EntryName): DetectedFileType {
   const ext = Path.getFileExtension(name);
 
   if (IMAGES.includes(ext)) {
@@ -104,6 +106,9 @@ function detectFileContentType(name: EntryName): DetectedFileType | undefined {
   if (TEXTS.includes(ext)) {
     return { type: FileContentType.Text, extension: ext };
   }
+  if (PRESENTATIONS.includes(ext)) {
+    return { type: FileContentType.Presentation, extension: ext };
+  }
   return { type: FileContentType.Unknown, extension: ext };
 }
 

client/src/parsec/mock_files/index.ts

@@ -10,6 +10,7 @@ export enum MockFileType {
   XLSX = 'XLSX',
   MP4 = 'MP4',
   MP3 = 'MP3',
+  PPTX = 'PPTX',
 }
 
 // Import content dynamically so it's not loaded if not needed
@@ -33,5 +34,7 @@ export async function getMockFileContent(type: MockFileType): Promise<Uint8Array
       return (await import('@/parsec/mock_files/mp4')).default;
     case MockFileType.MP3:
       return (await import('@/parsec/mock_files/mp3')).default;
+    case MockFileType.PPTX:
+      return (await import('@/parsec/mock_files/pptx')).default;
   }
 }

client/src/parsec/mock_files/pptx.ts

@@ -213,7 +213,9 @@ export class Cryptpad {
       // Tried to monkeypatch pushState, nop. There's no way to detect that
       // the iframe navigated, changing the main history enough that
       // history.back() is required, but not enough that `history.state` changes.
-      window.history.back();
+      if (window.history) {
+        window.history.back();
+      }
     } finally {
       window.clearTimeout(loadingTimeoutId);
       // Clean up global config reference