From af37d372abe5766a294131d5ada9bb30626ba4b3 Mon Sep 17 00:00:00 2001
From: rudouglas <ruairidouglas@gmail.com>
Date: Fri, 4 Jun 2021 00:06:37 +0100
Subject: [PATCH] fix: removed float and refactored creeateMenuItems

---
 src/components/InstallButton.js | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/src/components/InstallButton.js b/src/components/InstallButton.js
index a58532910..9d5619cee 100644
--- a/src/components/InstallButton.js
+++ b/src/components/InstallButton.js
@@ -5,13 +5,8 @@ import { css } from '@emotion/react';
 
 const sampleItems = new Array(10).fill().map((_, i) => i + 1);
 
-const createMenuItems = (items) => items.map((i) => <Dropdown.MenuItem>{i}</Dropdown.MenuItem>)
-  const menuItems = [];
-  for (const item of items) {
-    menuItems.push(<Dropdown.MenuItem>{item}</Dropdown.MenuItem>);
-  }
-  return menuItems;
-};
+const createMenuItems = (items) =>
+  items.map((item) => <Dropdown.MenuItem>{item}</Dropdown.MenuItem>);
 
 const createInstallLink = () => {
   return `https://one.newrelic.com/launcher/nr1-core.explorer`;
@@ -23,7 +18,6 @@ const InstallButton = ({ children, title, ...props }) => {
       css={css`
         display: flex;
         flex-direction: row;
-        float: right;
       `}
     >
       <Button
@@ -48,7 +42,7 @@ const InstallButton = ({ children, title, ...props }) => {
             padding: 5px;
           `}
         />
-        <Dropdown.Menu>{createMenuItems(children, sampleItems)}</Dropdown.Menu>
+        <Dropdown.Menu>{createMenuItems(sampleItems)}</Dropdown.Menu>
       </Dropdown>
     </div>
   );