From 9191533634a3a6ad250cdf779e5804ac7b80d9c9 Mon Sep 17 00:00:00 2001
From: Sebastian Malton <sebastian@malton.name>
Date: Thu, 22 Apr 2021 08:14:10 -0400
Subject: [PATCH] refactor AddRemoveButtons's renderButtons method to be just a
 filter map (#2210)

Signed-off-by: Sebastian Malton <sebastian@malton.name>
---
 .../add-remove-buttons/add-remove-buttons.tsx | 22 +++++++------------
 1 file changed, 8 insertions(+), 14 deletions(-)

diff --git a/src/renderer/components/add-remove-buttons/add-remove-buttons.tsx b/src/renderer/components/add-remove-buttons/add-remove-buttons.tsx
index 55cb745fb4..2651651b30 100644
--- a/src/renderer/components/add-remove-buttons/add-remove-buttons.tsx
+++ b/src/renderer/components/add-remove-buttons/add-remove-buttons.tsx
@@ -15,7 +15,8 @@ export interface AddRemoveButtonsProps extends React.HTMLAttributes<any> {
 export class AddRemoveButtons extends React.PureComponent<AddRemoveButtonsProps> {
   renderButtons() {
     const { onRemove, onAdd, addTooltip, removeTooltip } = this.props;
-    const buttons = [
+
+    return [
       {
         onClick: onRemove,
         className: "remove-button",
@@ -28,20 +29,13 @@ export class AddRemoveButtons extends React.PureComponent<AddRemoveButtonsProps>
         icon: "add",
         tooltip: addTooltip,
       },
-    ];
-
-    return buttons.map(button => {
-      if (!button.onClick) {
-        return null;
-      }
-      const { onClick, className, icon, tooltip } = button;
-
-      return (
-        <Button key={icon} big round primary onClick={onClick} className={className} tooltip={tooltip}>
-          <Icon material={icon}/>
+    ]
+      .filter(button => button.onClick)
+      .map(({ icon, ...props }) => (
+        <Button key={icon} big round primary {...props}>
+          <Icon material={icon} />
         </Button>
-      );
-    });
+      ));
   }
 
   render() {