Refactor unnecessary promise

Author: geoffrich

src/lib/ListItem.svelte

@@ -7,11 +7,9 @@
 
 	const transitionNextState = getPageTransitionTrigger();
 
-	async function handleClick(e) {
+	function handleClick() {
 		// TODO: this only works on click, not back/forward
-		const elm = e.target.closest('a');
-		// did this await make next wait for navigation?
-		await transitionNextState();
+		transitionNextState();
 	}
 </script>
 

src/lib/utils/use-page-transition.js

@@ -15,13 +15,10 @@ export const getPageTransitionTrigger = () => {
 			return null;
 		}
 
-		return new Promise((resolve) => {
-			const transition = document.createDocumentTransition();
-			transition.start(async () => {
-				resolve();
-				await new Promise((resolver) => {
-					transitionStore.set({ transition, resolver });
-				});
+		const transition = document.createDocumentTransition();
+		transition.start(async () => {
+			await new Promise((resolver) => {
+				transitionStore.set({ transition, resolver });
 			});
 		});
 	};