Merge pull request #594 from preactjs/iso-router-link-handler-race

Author: marvinhagemeister

router.js

@@ -63,7 +63,7 @@ export function LocationProvider(props) {
 		return { url, path, query: Object.fromEntries(u.searchParams), route, wasPush };
 	}, [url]);
 
-	useEffect(() => {
+	useLayoutEffect(() => {
 		addEventListener('click', route);
 		addEventListener('popstate', route);
 

test/router.test.js

@@ -429,7 +429,7 @@ describe('Router', () => {
 		await sleep(20);
 
 		scratch.querySelector('a[href="/foo#foo"]').click();
-		await sleep(100);
+		await sleep(20);
 		expect(Route).toHaveBeenCalledTimes(1);
 		expect(loc).toMatchObject({ url: '/foo#foo', path: '/foo' });
 		expect(pushState).toHaveBeenCalled();