Boost: Remove logic to reset popup when score changes (#35219)

* Remove logic to reset popup when score changes

This is faulty and causes more bugs than it fixes.

* add changelog

projects/plugins/boost/app/assets/src/js/features/speed-score/pop-out/pop-out.tsx

@@ -2,7 +2,7 @@ import { animated, useSpring } from '@react-spring/web';
 import CloseButton from '$features/ui/close-button/close-button';
 import styles from './pop-out.module.scss';
 import { __ } from '@wordpress/i18n';
-import { ReactNode, useEffect, useState } from 'react';
+import { ReactNode, useState } from 'react';
 import { Button } from '@wordpress/components';
 import { useDismissibleAlertState } from '$features/performance-history/lib/hooks';
 import { getRedirectUrl } from '@automattic/jetpack-components';
@@ -64,11 +64,6 @@ function PopOut( { scoreChange }: Props ) {
 	 */
 	const message = scoreChange && scoreChange < 0 ? slowerMessage : fasterMessage;
 
-	/*
-	 * Track the last message shown so we can detect when it changes.
-	 */
-	const [ lastMessage, setLastMessage ] = useState< ScoreChangeMessage | null >( null );
-
 	/*
 	 * Use datasync to track which score alerts have been dismissed.
 	 * Dismissed means that the user asked to never show us this alert again.
@@ -80,16 +75,6 @@ function PopOut( { scoreChange }: Props ) {
 	 */
 	const [ isClosed, setClose ] = useState( false );
 
-	/*
-	 * If the message has changed, reset the close state.
-	 */
-	useEffect( () => {
-		if ( message.id !== lastMessage?.id ) {
-			setLastMessage( message );
-			setClose( false );
-		}
-	}, [ message, lastMessage ] );
-
 	const hideAlert = () => setClose( true );
 
 	const animationStyles = useSpring( {

projects/plugins/boost/changelog/boost-react-speed-score-popout-revert

@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Revert speed score popout not showing "fix".
+
+