From 667b96ec330fee0cd18c7d041d579df9d673fae9 Mon Sep 17 00:00:00 2001
From: Brandon Presley <brandon@presley.nz>
Date: Sat, 24 Dec 2022 19:49:43 +1300
Subject: [PATCH] Start simplifying Switch.tsx

---
 LabelledButton.tsx | 25 ++++++++++++
 SettingsPage.tsx   | 97 ++++++++++++++++------------------------------
 Switch.tsx         | 49 ++++++++++++-----------
 3 files changed, 86 insertions(+), 85 deletions(-)
 create mode 100644 LabelledButton.tsx

diff --git a/LabelledButton.tsx b/LabelledButton.tsx
new file mode 100644
index 0000000..5a219cb
--- /dev/null
+++ b/LabelledButton.tsx
@@ -0,0 +1,25 @@
+import {View} from 'react-native'
+import {ITEM_PADDING} from './constants'
+import {Button, Subheading} from 'react-native-paper'
+
+export default function LabelledButton({
+  label,
+  onPress,
+  children,
+}: {
+  label?: string
+  onPress: () => void
+  children: string
+}) {
+  return (
+    <View
+      style={{
+        flexDirection: 'row',
+        alignItems: 'center',
+        paddingLeft: ITEM_PADDING,
+      }}>
+      <Subheading style={{width: 100}}>{label}</Subheading>
+      <Button onPress={onPress}>{children}</Button>
+    </View>
+  )
+}
diff --git a/SettingsPage.tsx b/SettingsPage.tsx
index a084056..8bf0edb 100644
--- a/SettingsPage.tsx
+++ b/SettingsPage.tsx
@@ -6,7 +6,7 @@ import {
 import {format} from 'date-fns'
 import {useCallback, useEffect, useMemo, useState} from 'react'
 import {Controller, useForm} from 'react-hook-form'
-import {DeviceEventEmitter, NativeModules, Platform, View} from 'react-native'
+import {NativeModules, Platform, View} from 'react-native'
 import DocumentPicker from 'react-native-document-picker'
 import {Dirs, FileSystem} from 'react-native-file-access'
 import {Button, Subheading} from 'react-native-paper'
@@ -16,6 +16,7 @@ import {AppDataSource} from './data-source'
 import {setRepo, settingsRepo} from './db'
 import {DrawerParamList} from './drawer-param-list'
 import DrawerHeader from './DrawerHeader'
+import LabelledButton from './LabelledButton'
 import {darkOptions, lightOptions, themeOptions} from './options'
 import Page from './Page'
 import Select from './Select'
@@ -41,16 +42,12 @@ export default function SettingsPage() {
 
   useEffect(() => {
     if (Object.keys(settings).length === 0) return
-    console.log(`${SettingsPage.name}.update`)
+    console.log(`${SettingsPage.name}.update`, {settings})
     settingsRepo.update({}, settings)
     setLightColor(settings.lightColor)
     setDarkColor(settings.darkColor)
     setTheme(settings.theme)
     if (!settings.alarm || ignoring) return
-    DeviceEventEmitter.emit('toast', {
-      value: 'Timers will now run after each set',
-      timeout: 4000,
-    })
     NativeModules.SettingsModule.ignoreBattery()
     setIgnoring(true)
   }, [settings, setDarkColor, setLightColor, setTheme, ignoring])
@@ -86,21 +83,9 @@ export default function SettingsPage() {
 
   const renderSwitch = useCallback(
     (key: keyof Settings) => (
-      <Controller
-        key={key}
-        name={key}
-        control={control}
-        render={({field: {onChange, value}}) => (
-          <Switch
-            value={value as boolean}
-            onPress={() => {
-              onChange(!value)
-            }}
-            onChange={onChange}>
-            {toSentenceCase(key)}
-          </Switch>
-        )}
-      />
+      <Switch control={control} name={key}>
+        {toSentenceCase(key)}
+      </Switch>
     ),
     [control],
   )
@@ -185,48 +170,34 @@ export default function SettingsPage() {
     toast('Database exported. Check downloads.')
   }, [])
 
-  const buttons = useMemo(
-    () => [
-      {
-        name: 'Alarm sound',
-        element: (
-          <View
-            key="alarm-sound"
-            style={{
-              flexDirection: 'row',
-              alignItems: 'center',
-              paddingLeft: ITEM_PADDING,
-            }}>
-            <Subheading style={{width: 100}}>Alarm sound</Subheading>
-            <Button onPress={changeSound}>{soundString || 'Default'}</Button>
-          </View>
-        ),
-      },
-      {
-        name: 'Export database',
-        element: (
-          <Button
-            key="export-db"
-            style={{alignSelf: 'flex-start'}}
-            onPress={exportDatabase}>
-            Export database
-          </Button>
-        ),
-      },
-      {
-        name: 'Import database',
-        element: (
-          <Button
-            key="import-db"
-            style={{alignSelf: 'flex-start'}}
-            onPress={() => setImporting(true)}>
-            Import database
-          </Button>
-        ),
-      },
-    ],
-    [changeSound, exportDatabase, soundString],
-  )
+  const buttons = [
+    {
+      name: 'Alarm sound',
+      element: (
+        <LabelledButton label="Alarm sound" onPress={changeSound}>
+          {soundString || 'Default'}
+        </LabelledButton>
+      ),
+    },
+    {
+      name: 'Export database',
+      element: (
+        <Button style={{alignSelf: 'flex-start'}} onPress={exportDatabase}>
+          Export database
+        </Button>
+      ),
+    },
+    {
+      name: 'Import database',
+      element: (
+        <Button
+          style={{alignSelf: 'flex-start'}}
+          onPress={() => setImporting(true)}>
+          Import database
+        </Button>
+      ),
+    },
+  ]
 
   return (
     <>
diff --git a/Switch.tsx b/Switch.tsx
index f0c1302..d37cc6d 100644
--- a/Switch.tsx
+++ b/Switch.tsx
@@ -1,36 +1,41 @@
+import {Control, Controller} from 'react-hook-form'
 import {Platform, Pressable} from 'react-native'
 import {Switch as PaperSwitch, Text, useTheme} from 'react-native-paper'
 import {MARGIN} from './constants'
 
 export default function Switch({
-  value,
-  onChange,
-  onPress,
+  control,
+  name,
   children,
 }: {
-  value?: boolean
-  onChange: (value: boolean) => void
-  onPress: () => void
+  name: string
+  control: Control<any, any>
   children: string
 }) {
   const {colors} = useTheme()
 
   return (
-    <Pressable
-      onPress={onPress}
-      style={{
-        flexDirection: 'row',
-        flexWrap: 'wrap',
-        alignItems: 'center',
-        marginBottom: Platform.OS === 'ios' ? MARGIN : null,
-      }}>
-      <PaperSwitch
-        color={colors.primary}
-        style={{marginRight: MARGIN}}
-        value={value}
-        onValueChange={onChange}
-      />
-      <Text>{children}</Text>
-    </Pressable>
+    <Controller
+      name={name}
+      control={control}
+      render={({field: {onChange, value}}) => (
+        <Pressable
+          onPress={() => onChange(!value)}
+          style={{
+            flexDirection: 'row',
+            flexWrap: 'wrap',
+            alignItems: 'center',
+            marginBottom: Platform.OS === 'ios' ? MARGIN : null,
+          }}>
+          <PaperSwitch
+            color={colors.primary}
+            style={{marginRight: MARGIN}}
+            value={value}
+            onValueChange={onChange}
+          />
+          <Text>{children}</Text>
+        </Pressable>
+      )}
+    />
   )
 }