fix: always wait for navigated event for title

When running with react-router
always wait for vaadin-navigated
event before updating the title.

Fixes #21171

Author: caalador

flow-server/src/main/java/com/vaadin/flow/component/internal/UIInternals.java

@@ -748,7 +748,7 @@ private String generateTitleScript() {
             // For react-router we should wait for navigation to finish
             // before updating the title.
             setTitleScript = String.format(
-                    "if(window.Vaadin.Flow.navigation) { window.addEventListener('vaadin-navigated', function(event) {%s}, {once:true}); }  else { %1$s }",
+                    "if(window.Vaadin.Flow.navigation) { window.addEventListener('vaadin-navigated', function(event) {%s}, {once:true}); }  else { %1$s console.trace();}",
                     setTitleScript);
         }
         return setTitleScript;

flow-server/src/main/resources/com/vaadin/flow/server/frontend/Flow.tsx

@@ -426,8 +426,6 @@ function Flow() {
 
     const vaadinNavigateEventHandler = useCallback(
         (event: CustomEvent<{ state: unknown; url: string; replace?: boolean; callback: boolean }>) => {
-            // @ts-ignore
-            window.Vaadin.Flow.navigation = true;
             // clean base uri away if for instance redirected to http://localhost/path/user?id=10
             // else the whole http... will be appended to the url see #19580
             const path = event.detail.url.startsWith(document.baseURI)
@@ -448,13 +446,22 @@ function Flow() {
         [navigate]
     );
 
-    useEffect(() => {
+  const flowNavigation = () => {
+    // @ts-ignore
+    window.Vaadin.Flow.navigation = true;
+  };
+
+  useEffect(() => {
+        // @ts-ignore
+        window.addEventListener("popstate", flowNavigation);
         // @ts-ignore
         window.addEventListener('vaadin-router-go', vaadinRouterGoEventHandler);
         // @ts-ignore
         window.addEventListener('vaadin-navigate', vaadinNavigateEventHandler);
 
         return () => {
+            // @ts-ignore
+            window.removeEventListener("popstate", flowNavigation);
             // @ts-ignore
             window.removeEventListener('vaadin-router-go', vaadinRouterGoEventHandler);
             // @ts-ignore