hendrik/lens
1
0
Fork 0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00
Code Issues Projects Releases Wiki Activity

refactor AddRemoveButtons's renderButtons method to be just a filter map (#2210)

Browse Source 
Signed-off-by: Sebastian Malton <sebastian@malton.name>
This commit is contained in:
Sebastian Malton 2021-04-22 08:14:10 -04:00 committed by GitHub
parent 21585d882d
commit 9191533634
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/renderer/components/add-remove-buttons/add-remove-buttons.tsx
View File

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