open-source-labs · mylee1995 · Jul 26, 2019 · Jul 26, 2019
diff --git a/extension/app/__tests__/action.test.jsx → src/app/__tests__/action.test.jsx b/extension/app/__tests__/action.test.jsx → src/app/__tests__/action.test.jsx
diff --git a/extension/app/components/Action.jsx → src/app/components/Action.jsx b/extension/app/components/Action.jsx → src/app/components/Action.jsx
diff --git a/extension/app/components/App.jsx → src/app/components/App.jsx b/extension/app/components/App.jsx → src/app/components/App.jsx
diff --git a/extension/app/components/Slider.jsx → src/app/components/Slider.jsx b/extension/app/components/Slider.jsx → src/app/components/Slider.jsx
diff --git a/extension/app/containers/ActionContainer.jsx → src/app/containers/ActionContainer.jsx b/extension/app/containers/ActionContainer.jsx → src/app/containers/ActionContainer.jsx
diff --git a/extension/app/containers/HeadContainer.jsx → src/app/containers/HeadContainer.jsx b/extension/app/containers/HeadContainer.jsx → src/app/containers/HeadContainer.jsx
diff --git a/extension/app/containers/MainContainer.jsx → src/app/containers/MainContainer.jsx b/extension/app/containers/MainContainer.jsx → src/app/containers/MainContainer.jsx
@@ -9,6 +9,7 @@ class MainContainer extends Component {
     super();
     this.state = {
       snapshots: [],
+      snapshotsTree: null,
       snapshotIndex: 0,
       currentIndex: null,
       port: null,
@@ -20,13 +21,12 @@ class MainContainer extends Component {
   }
 
   componentDidMount() {
+    console.log('componentDidMount');
     // open connection with background script
     const port = chrome.runtime.connect();
 
     // listen for a message containing snapshots from the background script
     port.onMessage.addListener((snapshots) => {
-      console.log('message from background script', snapshots);
-      console.log('snapshots', this.state.snapshots);
       const snapshotIndex = snapshots.length - 1;
 
       // set state with the information received from the background script
@@ -49,9 +49,11 @@ class MainContainer extends Component {
   }
 
   // change the snapshot index
-  // --> 1. affects the action that is highlighted
-  // --> 2. moves the slider
+  // this will change the state shown in the state container but won't change the DOM
   handleChangeSnapshot(snapshotIndex) {
+    // snapshotIndex
+    // --> 1. affects the action that is highlighted
+    // --> 2. moves the slider
     this.setState({ snapshotIndex });
   }
 
@@ -61,23 +63,27 @@ class MainContainer extends Component {
     let { currentIndex } = this.state;
     const payload = [];
 
+    // currentIndex is initialized to null
+    // currentIndex = null means that the user hasn't jumped yet
     currentIndex = currentIndex === null ? snapshots.length - 1 : currentIndex;
 
+    // if the user wants to jump backward
     if (currentIndex > snapshotIndex) {
       for (let i = currentIndex - 1; i >= snapshotIndex; i -= 1) {
         payload.push(snapshots[i]);
       }
     } else {
+      // if the user wants to jump forward
       for (let i = currentIndex + 1; i <= snapshotIndex; i += 1) {
         payload.push(snapshots[i]);
       }
     }
 
+    // currentIndex should be changed to index the user jumped to
     this.setState({ currentIndex: snapshotIndex });
 
-    console.log('payload', payload);
+    // send the arrays of snapshots to background script
     port.postMessage({ action: 'stepToSnap', payload });
-    // port.postMessage({ action: 'jumpToSnap', payload: snapshots[snapshotIndex] });
   }
 
   render() {

diff --git a/extension/app/containers/StateContainer.jsx → src/app/containers/StateContainer.jsx b/extension/app/containers/StateContainer.jsx → src/app/containers/StateContainer.jsx
diff --git a/extension/app/containers/TravelContainer.jsx → src/app/containers/TravelContainer.jsx b/extension/app/containers/TravelContainer.jsx → src/app/containers/TravelContainer.jsx
diff --git a/extension/app/index.js → src/app/index.js b/extension/app/index.js → src/app/index.js
diff --git a/extension/app/styles/styles.css → src/app/styles/styles.css b/extension/app/styles/styles.css → src/app/styles/styles.css
diff --git a/extension/background.js → src/extension/background.js b/extension/background.js → src/extension/background.js
diff --git a/extension/contentScript.js → src/extension/contentScript.js b/extension/contentScript.js → src/extension/contentScript.js
@@ -1,4 +1,4 @@
-console.log('contentScript injected');
+console.log('contentScript running');
 
 // listening for messages from npm package
 window.addEventListener('message', (msg) => {

diff --git a/extension/devtools.html → src/extension/devtools.html b/extension/devtools.html → src/extension/devtools.html
diff --git a/extension/devtools.js → src/extension/devtools.js b/extension/devtools.js → src/extension/devtools.js
diff --git a/extension/manifest.json → src/extension/manifest.json b/extension/manifest.json → src/extension/manifest.json
diff --git a/extension/panel.html → src/extension/panel.html b/extension/panel.html → src/extension/panel.html
@@ -9,7 +9,7 @@
 
   <body>
     <div id="root"></div>
-    <link rel="stylesheet" type="text/css" href="./app/styles/panel.css" />
+    <link rel="stylesheet" type="text/css" href="./styles/panel.css" />
     <script type="text/javascript" src="panel.js"></script>
     <script type="text/javascript" src="dist/app.bundle.js"></script>
   </body>

diff --git a/extension/panel.js → src/extension/panel.js b/extension/panel.js → src/extension/panel.js
diff --git a/extension/app/styles/panel.css → src/extension/styles/panel.css b/extension/app/styles/panel.css → src/extension/styles/panel.css
diff --git a/src/extension/tree.js b/src/extension/tree.js
@@ -0,0 +1,7 @@
+module.exports = function Graph(snapshot, parent) {
+  this.parent = parent;
+  this.snapshot = snapshot;
+  this.children = [];
+  this.add = (snapshot) => {};
+  this.remove = () => {};
+};
diff --git a/webpack.config.js b/webpack.config.js
@@ -3,11 +3,11 @@ const ChromeExtensionReloader = require('webpack-chrome-extension-reloader');
 
 const config = {
   entry: {
-    app: './extension/app/index.js',
-    background: './extension/background.js',
+    app: './src/app/index.js',
+    background: './src/extension/background.js',
   },
   output: {
-    path: path.resolve(__dirname, 'extension/dist'),
+    path: path.resolve(__dirname, 'src/extension/dist'),
     filename: '[name].bundle.js',
   },
   module: {
@@ -27,10 +27,7 @@ const config = {
       },
       {
         test: /\.css$/,
-        use: [
-          'style-loader',
-          'css-loader',
-        ],
+        use: ['style-loader', 'css-loader'],
       },
     ],
   },
@@ -39,12 +36,14 @@ const config = {
 
 module.exports = (env, argv) => {
   if (argv.mode === 'development') {
-    config.plugins.push(new ChromeExtensionReloader({
-      entries: {
-        contentScript: ['app'],
-        background: ['background'],
-      },
-    }));
+    config.plugins.push(
+      new ChromeExtensionReloader({
+        entries: {
+          contentScript: ['app'],
+          background: ['background'],
+        },
+      }),
+    );
   }
   return config;
 };