From 719560b66456bb22aea99f3924f9438714b15635 Mon Sep 17 00:00:00 2001 From: Brandon Presley Date: Wed, 21 Sep 2022 11:59:14 +1200 Subject: [PATCH] Hide rest minutes and seconds from edit set screen Feels like there are too many fields on this screen so we can just leave the defaults for people who don't care about this feature. --- SetForm.tsx | 33 +++--------------- .../en-US/images/phoneScreenshots/edit.png | Bin 85944 -> 119763 bytes 2 files changed, 4 insertions(+), 29 deletions(-) diff --git a/SetForm.tsx b/SetForm.tsx index be3fa8d..8cccf6b 100644 --- a/SetForm.tsx +++ b/SetForm.tsx @@ -1,7 +1,6 @@ import React, {useEffect, useRef, useState} from 'react'; -import {ScrollView, View} from 'react-native'; +import {ScrollView} from 'react-native'; import {Button} from 'react-native-paper'; -import {MARGIN} from './constants'; import MassiveInput from './MassiveInput'; import Set from './set'; import {getSets} from './set.service'; @@ -20,8 +19,6 @@ export default function SetForm({ const [weight, setWeight] = useState(set.weight.toString()); const [unit, setUnit] = useState(set.unit); const [uri, setUri] = useState(set.image); - const [minutes, setMinutes] = useState(set.minutes?.toString()); - const [seconds, setSeconds] = useState(set.seconds?.toString()); const [selection, setSelection] = useState({ start: 0, end: set.reps.toString().length, @@ -29,8 +26,6 @@ export default function SetForm({ const weightRef = useRef(null); const repsRef = useRef(null); const unitRef = useRef(null); - const minutesRef = useRef(null); - const secondsRef = useRef(null); useEffect(() => { console.log('SetForm.useEffect:', {uri, name: set.name}); @@ -49,8 +44,8 @@ export default function SetForm({ id: set.id, unit, image: uri, - minutes: Number(minutes ?? 3), - seconds: Number(seconds ?? 30), + minutes: Number(set.minutes ?? 3), + seconds: Number(set.seconds ?? 30), sets: set.sets ?? 3, }); }; @@ -91,35 +86,15 @@ export default function SetForm({ label="Unit" value={unit} onChangeText={setUnit} - onSubmitEditing={() => minutesRef.current?.focus()} innerRef={unitRef} /> {workouts && ( )} - {!set.id && ( - - secondsRef.current?.focus()} - /> - - - )}