Fix must-click-twice bug in mobile (switch list from hash to query)

Author: varunsingh87

components/JottingsControls/jottingsControl.js

@@ -29,18 +29,9 @@ export default function JottingsControl(props) {
 	}, [width]);
 
     useEffect(() => {
-        const onHashChangeStart = (url) => {
-			const newHash = url.split('#')[1];
-            console.log(`Path changing to ${newHash}`);
-			setActiveList(newHash || 'labels');
-        };
-
-        router.events.on("hashChangeStart", onHashChangeStart);
-
-        return () => {
-            router.events.off("hashChangeStart", onHashChangeStart);
-        };
-    }, [router.events]);
+        setActiveList(router.query.list || 'labels');
+
+    }, [router.query]);
 
 	const getJotsToShow = (response, jotType) => {
 		if (

components/JottingsControls/noteControl.js

@@ -28,7 +28,7 @@ export default function NoteControl({ notes }) {
 		urlService.setQueryToJottingInfo("note");
 	}, [notes]);
 
-	if (showNote && router.query.type == "note") {
+	if (showNote) {
 		return (
 			<article ref={showShareMenu ? ref : null} className={`${jottingsControl.fullJotting} ${jottingsControl.noteControl}`}>
 				<div

components/MobileNav/MobileNav.js

@@ -4,9 +4,9 @@ import Link from "next/link";
 export default function MobileNav() {
     return (
         <nav className={mobileNav.mobileNav}>
-            <Link href="#labels">Labels</Link>
-            <Link href="#notes">Notes</Link>
-            <Link href="#tasks">Tasks</Link>
+            <Link href="?list=labels">Labels</Link>
+            <Link href="?list=notes">Notes</Link>
+            <Link href="?list=tasks">Tasks</Link>
         </nav>
     );
 }