New: Integrate new excel viewer into preview (box#2)

* New: Add fake excel viewer and add it to docLoader * New: Finish the dummy-viewer * New: Can parse using sheetjs on the client * New: SheetJS prototype * New: Parse xlsx in the viewer * Fix: Eslint * New: Remove XLSX package * Fix: Remove lodash and fix following issues * Fix: Grids text overlap * Fix: install modules from public instead of jfrog * Fix: use meaningful filename
mxiao6 · Jul 10, 2018 · f27195e · f27195e
1 parent 103a037
commit f27195e
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 9 deletions.
diff --git a/build/webpack.config.js b/build/webpack.config.js
@@ -36,7 +36,7 @@ function updateConfig(conf, language, index) {
             annotations: ['box-annotations'],
             preview: [`${lib}/Preview.js`],
             csv: [`${lib}/viewers/text/BoxCSV.js`],
-            excel: [`${lib}/viewers/doc/BoxExcel.js`]
+            excel: [`${lib}/viewers/doc/BoxExcel`]
         },
         output: {
             path: path.resolve('dist', version, language),

diff --git a/src/lib/viewers/doc/BoxExcel.js → src/lib/viewers/doc/BoxExcel/index.js b/src/lib/viewers/doc/BoxExcel.js → src/lib/viewers/doc/BoxExcel/index.js
@@ -1,6 +1,6 @@
 import React from 'react';
 import { render, unmountComponentAtNode } from 'react-dom';
-import DataGrids from './BoxExcel/dataGrids';
+import Workbook from './workbook';
 
 class BoxExcel {
     /**
@@ -36,7 +36,7 @@ class BoxExcel {
      * @private
      */
     renderExcel() {
-        this.gridComponent = render(<DataGrids workbook={this.data} />, this.excelEl);
+        this.gridComponent = render(<Workbook workbook={this.data} />, this.excelEl);
     }
 }
 

diff --git a/src/lib/viewers/doc/BoxExcel/virtualGrid.js → src/lib/viewers/doc/BoxExcel/sheet.js b/src/lib/viewers/doc/BoxExcel/virtualGrid.js → src/lib/viewers/doc/BoxExcel/sheet.js
@@ -74,7 +74,7 @@ const styles = {
     focusedDiv: {}
 };
 
-class VirtualGrid extends Component {
+class Sheet extends Component {
     /* eslint-disable */
     static propTypes = {
         sheet: PropTypes.object.isRequired,
@@ -528,4 +528,4 @@ class VirtualGrid extends Component {
     }
 }
 
-export default VirtualGrid;
+export default Sheet;
diff --git a/src/lib/viewers/doc/BoxExcel/dataGrids.js → src/lib/viewers/doc/BoxExcel/workbook.js b/src/lib/viewers/doc/BoxExcel/dataGrids.js → src/lib/viewers/doc/BoxExcel/workbook.js
@@ -2,9 +2,9 @@ import React, { PureComponent } from 'react';
 import Tab from 'box-react-ui/lib/components/tab-view/Tab';
 import TabView from 'box-react-ui/lib/components/tab-view/TabView';
 import PropTypes from 'prop-types';
-import VirtualGrid from './virtualGrid';
+import Sheet from './sheet';
 
-class DataGrids extends PureComponent {
+class Workbook extends PureComponent {
     static propTypes = {
         // eslint-disable-next-line
         workbook: PropTypes.object.isRequired
@@ -24,7 +24,7 @@ class DataGrids extends PureComponent {
                 {nonHiddenSheetsNames.map((name) => {
                     return (
                         <Tab key={name} title={name}>
-                            <VirtualGrid sheet={workbook.Sheets[name]} views={wbSettings.Views} theme={colorScheme} />
+                            <Sheet sheet={workbook.Sheets[name]} views={wbSettings.Views} theme={colorScheme} />
                         </Tab>
                     );
                 })}
@@ -33,4 +33,4 @@ class DataGrids extends PureComponent {
     }
 }
 
-export default DataGrids;
+export default Workbook;