fixes issue#66

Author: romitrivigo

client/src/pages/Search/Global.jsx

@@ -239,12 +239,6 @@ export default function SearchGlobal({ keyword }) {
         setUsers(data);
         setTotalResults(Number(headers["x-total"]));
         setTotalPages(Number(headers["x-total-pages"]));
-        if (window) {
-          window.scrollTo({
-            top: 0,
-            behavior: "smooth",
-          });
-        }
       }
     })();
 
@@ -262,6 +256,19 @@ export default function SearchGlobal({ keyword }) {
     setWindowWidth(window.innerWidth);
   };
 
+  /**
+   * Sets the new page number and gets the new set of users
+   * @param {Number} newPageNumber The new page number
+   */
+  const onChangePage = async (newPageNumber) => {
+    if (window) {
+      window.scrollTo({
+        top: 0,
+      });
+    }
+    searchContext.changePageNumber(newPageNumber);
+  };
+
   return (
     <>
       <div className={style.sideMenu}>
@@ -341,7 +348,11 @@ export default function SearchGlobal({ keyword }) {
             })}
           </div>
           <div>
-            <Pagination currentPage={page} numPages={totalPages} />
+            <Pagination
+              currentPage={page}
+              numPages={totalPages}
+              onChangePage={onChangePage}
+            />
           </div>
         </div>
       )}

client/src/pages/Search/Groups.jsx

@@ -129,14 +129,13 @@ export default function SearchGroups() {
    * @param {Number} newPageNumber The new page number
    */
   const onChangePage = async (newPageNumber) => {
-    setPage(newPageNumber);
-    await getMembersInGroup(selectedGroup.id, newPageNumber);
     if (window) {
       window.scrollTo({
         top: 0,
-        behavior: "smooth",
       });
     }
+    setPage(newPageNumber);
+    await getMembersInGroup(selectedGroup.id, newPageNumber);
   };
 
   return (