From 73e0890b3be481b585b0b728e94f5e2e2f26aa88 Mon Sep 17 00:00:00 2001
From: Arthur Jamet <60505370+Arthi-chaud@users.noreply.github.com>
Date: Sat, 27 May 2023 18:42:38 +0100
Subject: [PATCH] Front: Play View: Better toasts for score messages (#207)

---
 front/views/PlayView.tsx | 42 +++++++++++++++++++++++++++++++++-------
 1 file changed, 35 insertions(+), 7 deletions(-)

diff --git a/front/views/PlayView.tsx b/front/views/PlayView.tsx
index 6577e10..3650dc9 100644
--- a/front/views/PlayView.tsx
+++ b/front/views/PlayView.tsx
@@ -1,7 +1,7 @@
 import React, { useEffect, useRef, useState } from 'react';
-import { SafeAreaView, Platform } from 'react-native';
+import { SafeAreaView, Platform, Animated } from 'react-native';
 import * as ScreenOrientation from 'expo-screen-orientation';
-import {  Box, Center, Column, Progress, Text, Row, View, useToast, Icon } from 'native-base';
+import {  Box, Center, Column, Progress, Text, Row, View, useToast, Icon, HStack } from 'native-base';
 import IconButton from '../components/IconButton';
 import { Ionicons, MaterialCommunityIcons } from "@expo/vector-icons";
 import { RouteProps, useNavigation } from "../Navigation";
@@ -13,16 +13,22 @@ import VirtualPiano from '../components/VirtualPiano/VirtualPiano';
 import { strToKey, keyToStr, Note } from '../models/Piano';
 import { useSelector } from 'react-redux';
 import { RootState } from '../state/Store';
-import { translate } from '../i18n/i18n';
+import { Translate, translate } from '../i18n/i18n';
 import { ColorSchemeType } from 'native-base/lib/typescript/components/types';
 import { useStopwatch } from "react-use-precision-timer";
 import PartitionView from '../components/PartitionView';
+import TextButton from '../components/TextButton';
 
 type PlayViewProps = {
 	songId: number,
 	type: 'practice' | 'normal'
 }
 
+type ScoreMessage = {
+	content: string,
+	color?: ColorSchemeType
+}
+
 
 // this a hot fix this should be reverted soon
 let scoroBaseApiUrl = Constants.manifest?.extra?.scoroUrl;
@@ -49,6 +55,7 @@ const PlayView = ({ songId, type, route }: RouteProps<PlayViewProps>) => {
 	const navigation = useNavigation();
 	const song = useQuery(['song', songId], () => API.getSong(songId), { staleTime: Infinity });
 	const toast = useToast();
+	const [lastScoreMessage, setLastScoreMessage] = useState<ScoreMessage>();
 	const webSocket = useRef<WebSocket>();
 	const [paused, setPause] = useState<boolean>(true);
 	const stopwatch = useStopwatch();
@@ -56,6 +63,7 @@ const PlayView = ({ songId, type, route }: RouteProps<PlayViewProps>) => {
 	const [time, setTime] = useState(0);
 	const [partitionRendered, setPartitionRendered] = useState(false); // Used to know when partitionview can render
 	const [score, setScore] = useState(0); // Between 0 and 100
+	const fadeAnim = useRef(new Animated.Value(0)).current;
 	const musixml = useQuery(["musixml", songId], () =>
 		API.getSongMusicXML(songId).then((data) => new TextDecoder().decode(data)),
 		{ staleTime: Infinity }
@@ -131,10 +139,10 @@ const PlayView = ({ songId, type, route }: RouteProps<PlayViewProps>) => {
 					formattedMessage = translate(data[data.type]);
 					switch (data[data.type]) {
 						case 'perfect':
-							messageColor = 'fuchsia';
+							messageColor = 'green';
 							break;
 						case 'great':
-							messageColor = 'green';
+							messageColor = 'blue';
 							break;
 						case 'short':
 						case 'long':
@@ -144,13 +152,14 @@ const PlayView = ({ songId, type, route }: RouteProps<PlayViewProps>) => {
 						case 'too short':
 						case 'too long':
 						case 'wrong':
-							messageColor = 'grey';
+							messageColor = 'trueGray';
 							break;
 						default:
 							break;
 					}
 				}
-				toast.show({ description: formattedMessage, placement: 'top', colorScheme: messageColor ?? 'secondary' });
+				setLastScoreMessage({ content: formattedMessage, color: messageColor });
+				// toast.show({ description: formattedMessage, placement: 'top', colorScheme: messageColor ?? 'secondary' });
 			} catch (e) {
 				console.log(e);
 			}
@@ -192,6 +201,16 @@ const PlayView = ({ songId, type, route }: RouteProps<PlayViewProps>) => {
 			clearInterval(interval);
 		}
 	}, []);
+	useEffect(() => {
+		if (lastScoreMessage) {
+			fadeAnim.setValue(1);
+			Animated.timing(fadeAnim, {
+				toValue: 0,
+				duration: 3000,
+				useNativeDriver: true,
+			}).start();
+		}
+	}, [lastScoreMessage]);
 	useEffect(() => {
 		// Song.data is updated on navigation.navigate (do not know why)
 		// Hotfix to prevent midi setup process from reruning on game end
@@ -208,6 +227,15 @@ const PlayView = ({ songId, type, route }: RouteProps<PlayViewProps>) => {
 	}
 	return (
 		<SafeAreaView style={{ flexGrow: 1, flexDirection: 'column' }}>
+			<HStack width="100%" justifyContent="center" p={3} style={{ position: 'absolute', top: 1 }}>
+				<Animated.View style={{ opacity: fadeAnim }}>
+					<TextButton
+						disabled
+						translate={{ translationKey: lastScoreMessage?.content ?? '' }}
+						colorScheme={lastScoreMessage?.color} rounded='sm'
+					/>
+				</Animated.View>
+			</HStack>
 			<View style={{ flexGrow: 1, justifyContent: 'center' }}>
 				<PartitionView file={musixml.data}
 					onPartitionReady={() => setPartitionRendered(true)}