Merge pull request #11 from oslabs-beta/ui-bugs

Ui bugs

Author: nusanam

src/app/components/Action.jsx

@@ -3,8 +3,9 @@ import PropTypes from 'prop-types';
 
 import { changeView, changeSlider } from '../actions/actions';
 
-// BUG FIX: 
-  // changeSlider should also respond to the click event on the div
+// Launch Feature: Figure out changeView vs changeSlider
+// Should we make the btn bigger instead and keep the functionality? 
+// div onclick event triggers the changeView method 
 
 const Action = props => {
   const {

src/app/components/Diff.jsx

@@ -5,10 +5,8 @@ import ReactHtmlParser from 'react-html-parser';
 
 import { useStoreContext } from '../store';
 
-// FIX: Update the div copy to something more explanatory 
-
 function Diff({ snapshot, show }) {
-  const [mainState] = useStoreContext();
+  const [ mainState ] = useStoreContext();
   const { currentTab, tabs } = mainState;
   const { snapshots, viewIndex, sliderIndex } = tabs[currentTab];
   let previous;
@@ -26,7 +24,7 @@ function Diff({ snapshot, show }) {
   if (show) formatters.html.showUnchanged();
   else formatters.html.hideUnchanged();
 
-  if (previous === undefined || delta === undefined) return <div> states are equal </div>;
+  if (previous === undefined || delta === undefined) return <div> No state change detected. </div>;
   return (
     <div>
       { ReactHtmlParser(html) }

src/app/components/SwitchApp.jsx

@@ -4,7 +4,7 @@ import { useStoreContext } from '../store';
 import { setTab } from '../actions/actions';
 
 
-// BUG FIX: Fix the dropdown styling to make it more distinguishable
+// Launch Feature: Fix the dropdown styling to make it more distinguishable
 
 const SwitchAppDropdown = () => {
   const [{ currentTab, tabs }, dispatch] = useStoreContext();

src/app/containers/ActionContainer.jsx

@@ -5,10 +5,7 @@ import Action from '../components/Action';
 import { emptySnapshots } from '../actions/actions';
 import { useStoreContext } from '../store';
 
-// MVP Feature: Include a dropdown functionality 
-// to show stateful/functional/Context API differentiation
-// May want to add another child component to the container
-
+// Launch: render a dropdown filterable list 
 function ActionContainer() {
   const [{ tabs, currentTab }, dispatch] = useStoreContext();
   const { snapshots, sliderIndex, viewIndex } = tabs[currentTab];