avoid lambda props

This fixes the simple cases of callback props using lambda functions.
This will prevent rerendering in some cases since a new callback
function is not created on each call.

src/about/AboutDialog.test.tsx

@@ -10,7 +10,7 @@ import AboutDialog from './AboutDialog';
 it('should close when the button is clicked', () => {
     const close = jest.fn();
 
-    const [dialog] = testRender(<AboutDialog isOpen={true} onClose={() => close()} />);
+    const [dialog] = testRender(<AboutDialog isOpen={true} onClose={close} />);
 
     userEvent.click(dialog.getByLabelText('Close'));
 

src/about/AboutDialog.tsx

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: MIT
-// Copyright (c) 2021 The Pybricks Authors
+// Copyright (c) 2021-2022 The Pybricks Authors
 
 // The about dialog
 
@@ -33,7 +33,7 @@ const AboutDialog: React.FunctionComponent<AboutDialogProps> = (props) => {
         <Dialog
             title={`Pybricks v${firmwareVersion} (${appName} v${appVersion})`}
             isOpen={props.isOpen}
-            onClose={() => props.onClose()}
+            onClose={props.onClose}
         >
             <div className={Classes.DIALOG_BODY}>
                 <div className="pb-about-icon">

src/explorer/Explorer.tsx

@@ -55,7 +55,7 @@ const ActionButton: React.VoidFunctionComponent<ActionButtonProps> = (props) =>
             title={i18n.translate(props.toolTipId, props.toolTipReplacements)}
             disabled={props.disabled}
             onMouseDown={preventFocusOnClick}
-            onClick={() => props.onClick()}
+            onClick={props.onClick}
         />
     );
 };

src/explorer/NewFileWizard.tsx

@@ -51,15 +51,15 @@ const NewFileWizard: React.VoidFunctionComponent<NewFileWizardProps> = (props) =
             isOpen={props.isOpen}
             onOpening={() => setFileName('')}
             onOpened={() => fileNameInputRef.current?.focus()}
-            onClose={() => props.onClose()}
+            onClose={props.onClose}
         >
             <div className={Classes.DIALOG_BODY}>
                 <FileNameFormGroup
                     fileName={fileName}
                     fileExtension={pythonFileExtension}
                     inputRef={fileNameInputRef}
-                    onChange={(n) => setFileName(n)}
-                    onValidation={(r) => setFileNameValidation(r)}
+                    onChange={setFileName}
+                    onValidation={setFileNameValidation}
                 />
                 <FormGroup label={i18n.translate(NewFileWizardStringId.SmartHubLabel)}>
                     <RadioGroup

src/explorer/RenameFileDialog.tsx

@@ -52,8 +52,8 @@ const RenameFileDialog: React.VoidFunctionComponent<RenameFileDialogProps> = (
                     fileName={newName}
                     fileExtension={extension}
                     inputRef={inputRef}
-                    onChange={(n) => setNewName(n)}
-                    onValidation={(r) => setResult(r)}
+                    onChange={setNewName}
+                    onValidation={setResult}
                 />
             </div>
             <div className={Classes.DIALOG_FOOTER}>

src/settings/SettingsDrawer.tsx

@@ -92,7 +92,7 @@ const SettingsDrawer: React.FunctionComponent<SettingsProps> = (props) => {
             icon="cog"
             size={Drawer.SIZE_SMALL}
             title={i18n.translate(SettingsStringId.Title)}
-            onClose={() => props.onClose()}
+            onClose={props.onClose}
         >
             <div className={Classes.DRAWER_BODY}>
                 <div className={Classes.DIALOG_BODY}>

src/toolbar/ActionButton.tsx

@@ -106,7 +106,7 @@ const ActionButton: React.FC<ActionButtonProps> = (props) => {
                     {...tooltipTargetProps}
                     intent={Intent.PRIMARY}
                     onMouseDown={preventFocusOnClick}
-                    onClick={() => props.onAction()}
+                    onClick={props.onAction}
                     disabled={props.enabled === false}
                     style={
                         props.enabled === false ? { pointerEvents: 'none' } : undefined