jumping now moves slider and jumping no longer changes state container

Author: rydang

src/app/components/Action.jsx

@@ -14,7 +14,10 @@ const Action = (props) => {
       <div className="action-component-text">{index}</div>
       <div
         className="jump-button"
-        onClick={() => handleJumpSnapshot(index)}
+        onClick={(e) => {
+          e.stopPropagation();
+          handleJumpSnapshot(index);
+        }}
         tabIndex={index}
         role="button"
       >
@@ -24,9 +27,11 @@ const Action = (props) => {
   );
 };
 
-// Action.propTypes = {
-//   selected: PropTypes.bool,
-//   index: PropTypes.number,
-// };
+Action.propTypes = {
+  selected: PropTypes.bool.isRequired,
+  index: PropTypes.number.isRequired,
+  handleChangeSnapshot: PropTypes.func.isRequired,
+  handleJumpSnapshot: PropTypes.func.isRequired,
+};
 
 export default Action;

src/app/containers/MainContainer.jsx

@@ -40,15 +40,14 @@ class MainContainer extends Component {
       const { action, payload } = message;
       switch (action) {
         case 'sendSnapshots': {
-          const viewIndex = payload.length - 1;
-          const sliderIndex = viewIndex;
+          const sliderIndex = payload.length - 1;
           // set state with the information received from the background script
-          this.setState({ snapshots: payload, viewIndex, sliderIndex });
+          this.setState({ snapshots: payload, sliderIndex });
           break;
         }
         case 'initialConnectSnapshots': {
           const { snapshots, mode } = payload;
-          const viewIndex = snapshots.length - 1;
+          const viewIndex = 0;
           const sliderIndex = viewIndex;
           this.setState({ snapshots, mode, viewIndex, sliderIndex });
           break;