From a26babc7924aa87f0a90343253e8213472892baf Mon Sep 17 00:00:00 2001
From: m5r <mokht@rmi.al>
Date: Fri, 13 Aug 2021 19:55:05 +0800
Subject: [PATCH] small qol changes

---
 app/phone-calls/hooks/use-make-call.ts          |  2 +-
 app/phone-calls/pages/keypad.tsx                | 17 +++++++++++++----
 .../pages/outgoing-call/[recipient].tsx         |  9 +++++----
 3 files changed, 19 insertions(+), 9 deletions(-)

diff --git a/app/phone-calls/hooks/use-make-call.ts b/app/phone-calls/hooks/use-make-call.ts
index a90d330..99ec76b 100644
--- a/app/phone-calls/hooks/use-make-call.ts
+++ b/app/phone-calls/hooks/use-make-call.ts
@@ -87,7 +87,7 @@ export default function useMakeCall(recipient: string) {
 
 		if (device) {
 			device.disconnectAll();
-			// device.destroy();
+			device.destroy();
 		}
 	}
 
diff --git a/app/phone-calls/pages/keypad.tsx b/app/phone-calls/pages/keypad.tsx
index 953b670..61ac022 100644
--- a/app/phone-calls/pages/keypad.tsx
+++ b/app/phone-calls/pages/keypad.tsx
@@ -1,5 +1,5 @@
 import type { BlitzPage } from "blitz";
-import { Link, Routes } from "blitz";
+import { Link, Routes, useRouter } from "blitz";
 import { useRef } from "react";
 import { atom, useAtom } from "jotai";
 import { FontAwesomeIcon } from "@fortawesome/react-fontawesome";
@@ -11,6 +11,7 @@ import useRequireOnboarding from "../../core/hooks/use-require-onboarding";
 
 const KeypadPage: BlitzPage = () => {
 	useRequireOnboarding();
+	const router = useRouter();
 	const [phoneNumber, setPhoneNumber] = useAtom(phoneNumberAtom);
 	const pressBackspace = useAtom(pressBackspaceAtom)[1];
 	const timeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null);
@@ -31,6 +32,7 @@ const KeypadPage: BlitzPage = () => {
 	};
 	const onDigitPressProps = (digit: string) => ({
 		onPress() {
+			// navigator.vibrate(1); // removed in webkit
 			pressDigit(digit);
 		},
 	});
@@ -43,12 +45,19 @@ const KeypadPage: BlitzPage = () => {
 
 			<Keypad onDigitPressProps={onDigitPressProps} onZeroPressProps={onZeroPressProps}>
 				<Link href={Routes.OutgoingCall({ recipient: encodeURI(phoneNumber) })}>
-					<a
-						onClick={() => setPhoneNumber("")}
+					<button
+						onClick={async () => {
+							if (phoneNumber === "") {
+								// TODO setPhoneNumber(lastCallRecipient);
+							}
+
+							await router.push(Routes.OutgoingCall({ recipient: encodeURI(phoneNumber) }));
+							setPhoneNumber("");
+						}}
 						className="cursor-pointer select-none col-start-2 h-12 w-12 flex justify-center items-center mx-auto bg-green-800 rounded-full"
 					>
 						<FontAwesomeIcon className="w-6 h-6" icon={faPhone} color="white" size="lg" />
-					</a>
+					</button>
 				</Link>
 				<div className="cursor-pointer select-none m-auto" onClick={pressBackspace}>
 					<FontAwesomeIcon className="w-6 h-6" icon={faBackspace} size="lg" />
diff --git a/app/phone-calls/pages/outgoing-call/[recipient].tsx b/app/phone-calls/pages/outgoing-call/[recipient].tsx
index 157c5c7..46bb20d 100644
--- a/app/phone-calls/pages/outgoing-call/[recipient].tsx
+++ b/app/phone-calls/pages/outgoing-call/[recipient].tsx
@@ -22,7 +22,7 @@ const OutgoingCall: BlitzPage = () => {
 	}, [call.state]);
 
 	useRequireOnboarding();
-	const phoneNumber = useAtom(phoneNumberAtom)[0];
+	const [phoneNumber, setPhoneNumber] = useAtom(phoneNumberAtom);
 	const pressDigit = useAtom(pressDigitAtom)[1];
 	const onDigitPressProps = useMemo(
 		() => (digit: string) => ({
@@ -37,11 +37,12 @@ const OutgoingCall: BlitzPage = () => {
 	const hangUp = useMemo(
 		() => () => {
 			call.hangUp();
+			setPhoneNumber("");
 
 			// return router.replace(Routes.KeypadPage());
 			return router.push(Routes.KeypadPage());
 		},
-		[call, router],
+		[call, router, setPhoneNumber],
 	);
 
 	return (
@@ -56,12 +57,12 @@ const OutgoingCall: BlitzPage = () => {
 			</div>
 
 			<Keypad onDigitPressProps={onDigitPressProps} onZeroPressProps={onDigitPressProps("0")}>
-				<div
+				<button
 					onClick={hangUp}
 					className="cursor-pointer select-none col-start-2 h-12 w-12 flex justify-center items-center mx-auto bg-red-800 rounded-full"
 				>
 					<FontAwesomeIcon className="w-6 h-6" icon={faPhone} color="white" size="lg" />
-				</div>
+				</button>
 			</Keypad>
 		</div>
 	);