Refactor components and fix minor details

client/components/Dropdown.css

@@ -7,7 +7,6 @@
   border: 1px solid #aaa;
   border-radius: 4px;
   display: block;
-  flex: 1;
   width: 100%;
   color: #7f7f7f;
   height: 27px;

client/components/Dropdown.jsx

@@ -1,5 +1,4 @@
 import PureComponent from '../lib/PureComponent';
-import DropdownOption from './DropdownOption';
 
 import s from './Dropdown.css';
 import {store} from '../store';
@@ -22,9 +21,9 @@ export default class Dropdown extends PureComponent {
         </div>
         <div>
           <select className={s.select} id={label} name={label} onChange={this.handleSelection}>
-            <DropdownOption value={DEFAULT_DROPDOWN_SELECTION}/>
+            <option value={DEFAULT_DROPDOWN_SELECTION}>{DEFAULT_DROPDOWN_SELECTION}</option>
             {options.map(option =>
-              <DropdownOption value={option}/>
+              <option key={option} value={option}>{option}</option>
             )}
           </select>
         </div>

src/analyzer.js

@@ -216,7 +216,7 @@ function createModulesTree(modules) {
 }
 
 function getChunkToInitialByEntrypoint(bundleStats) {
-  if (bundleStats === null || bundleStats === undefined) {
+  if (bundleStats == null) {
     return {};
   }
   const chunkToEntrypointInititalMap = {};

src/template.js

@@ -63,4 +63,4 @@ function renderViewer({title, enableWebSocket, chartData, entrypoints, defaultSi
     </script>
   </body>
 </html>`;
-}
+}

src/utils.js

@@ -62,4 +62,4 @@ exports.open = function (uri, logger) {
   } catch (err) {
     logger.debug(`Opener failed to open "${uri}":\n${err}`);
   }
-};
+};

src/viewer.js

@@ -198,4 +198,4 @@ function getEntrypoints(bundleStats) {
     return [];
   }
   return Object.values(bundleStats.entrypoints || {}).map(entrypoint => entrypoint.name);
-}
+}

test/utils.js

@@ -56,4 +56,4 @@ describe('createAssetsFilter', function () {
     expect(filter('bar-foo')).to.equal(true);
   });
 
-});
+});