way better experience searching for users

This commit is contained in:
Arjun Patel
2022-02-06 14:35:27 -08:00
parent 8a56a69a52
commit af6c3373cc
2 changed files with 36 additions and 29 deletions
+4 -1
View File
@@ -28,7 +28,10 @@ export default class Conversation {
this.name = _name; this.name = _name;
this.createdByUserId = _createdByUserId; this.createdByUserId = _createdByUserId;
this.activeMembers = _initialUsers; this.activeMembers = _initialUsers;
this.tldr = _tldr;
if (_tldr) {
this.tldr = _tldr;
}
} }
} }
@@ -1,6 +1,11 @@
import { User } from "@nirvana/common/models/user"; import { User } from "@nirvana/common/models/user";
import { RefObject, useEffect, useMemo, useRef, useState } from "react"; import { RefObject, useEffect, useMemo, useRef, useState } from "react";
import { FaPlus, FaRegTimesCircle, FaSearch } from "react-icons/fa"; import {
FaArrowRight,
FaPlus,
FaRegTimesCircle,
FaSearch,
} from "react-icons/fa";
import { usersIndex } from "../../services/algoliaSearchService"; import { usersIndex } from "../../services/algoliaSearchService";
import SimpleLoadingDot from "../Loading/SimpleLoadingDot"; import SimpleLoadingDot from "../Loading/SimpleLoadingDot";
import SimpleUserDetailsRow from "../UserDetails/SimpleUserDetailsRow"; import SimpleUserDetailsRow from "../UserDetails/SimpleUserDetailsRow";
@@ -17,6 +22,7 @@ import { QueryRoutes, Routes } from "@nirvana/common/helpers/routes";
import { useRouter } from "next/router"; import { useRouter } from "next/router";
import { useRecoilState } from "recoil"; import { useRecoilState } from "recoil";
import { useAuth } from "../../contexts/authContext"; import { useAuth } from "../../contexts/authContext";
import { HotKeys, KeyMap } from "react-hotkeys";
const searchDebounceMsTime = 3000; const searchDebounceMsTime = 3000;
@@ -44,21 +50,21 @@ export default function CreateConversation() {
input.current?.focus(); input.current?.focus();
}, []); }, []);
const handleChangeUserSearchInput = async (event) => { const handleUserSearch = async (event) => {
const newInput = event?.target?.value;
console.log("searching for : " + newInput);
setUserSearchInput(newInput);
// no search if empty input // no search if empty input
if (!newInput) { if (!userSearchInput) {
setIsLoading(false); setIsLoading(false);
console.warn("no valid input to search...no point searching"); console.warn("no valid input to search...no point searching");
return false; return false;
} }
console.log("searching for : " + userSearchInput);
try { try {
// do the search on the search value // do the search on the search value
const searchedUsers: SearchResponse = await usersIndex.search(newInput); const searchedUsers: SearchResponse = await usersIndex.search(
userSearchInput
);
console.log(searchedUsers.hits); console.log(searchedUsers.hits);
if (searchedUsers.hits) { if (searchedUsers.hits) {
@@ -79,19 +85,6 @@ export default function CreateConversation() {
setIsLoading(false); setIsLoading(false);
}; };
const debounceHandler = useMemo(
() => debounce(handleChangeUserSearchInput, searchDebounceMsTime),
[]
);
// Stop the invocation of the debounced function
// after unmounting
useEffect(() => {
return () => {
debounceHandler.cancel();
};
}, []);
const selectUser = (addUser: User) => { const selectUser = (addUser: User) => {
if (addUser.id == currUser!.uid) { if (addUser.id == currUser!.uid) {
toast.error("You cannot add yourself, silly!"); toast.error("You cannot add yourself, silly!");
@@ -165,7 +158,8 @@ export default function CreateConversation() {
const newConversation = new Conversation( const newConversation = new Conversation(
currUser!.uid, currUser!.uid,
conversationName, conversationName,
arrActiveMembers arrActiveMembers,
tldr
); );
console.log(members); console.log(members);
@@ -188,7 +182,7 @@ export default function CreateConversation() {
}; };
return ( return (
<div className="mx-auto my-auto w-96 flex flex-col"> <div className="mx-auto my-auto max-w-screen-sm w-full flex flex-col">
{/* <Select {/* <Select
placeholder={"Look up emails, names, etc."} placeholder={"Look up emails, names, etc."}
tagRender={tagRender} tagRender={tagRender}
@@ -203,20 +197,30 @@ export default function CreateConversation() {
{/* search bar that debounces and has a loading button while debouncing */} {/* search bar that debounces and has a loading button while debouncing */}
<span className="flex flex-row items-center space-x-2 mb-5"> <span className="relative bg-slate-50 p-2 rounded-lg flex flex-row items-center space-x-2 mb-5">
<FaSearch className="text-slate-300 text-lg" /> <FaSearch className="text-slate-300 text-lg" />
<input <input
className="p-2 flex-1 rounded bg-slate-50 border placeholder-slate-300" onKeyPress={(e) => {
if (e.key === "Enter") {
handleUserSearch(e);
}
}}
className="p-2 flex-1 rounded bg-slate-50 placeholder-slate-300 focus:outline-none"
placeholder="look up an email address or name" placeholder="look up an email address or name"
ref={input} ref={input}
onChange={(e) => { onChange={(e) => {
setIsLoading(true); setIsLoading(true);
debounceHandler(e); setUserSearchInput(e.target.value);
}} }}
/> />
{isLoading && <SimpleLoadingDot />} {userSearchInput && (
<span className="text-slate-300 flex flex-row items-center space-x-2">
<FaArrowRight />
<span>enter to search</span>
</span>
)}
</span> </span>
{/* show search results nicely like tailwind website */} {/* show search results nicely like tailwind website */}
@@ -313,7 +317,7 @@ export default function CreateConversation() {
<input <input
className="p-3 rounded mt-2 bg-slate-50 border placeholder-slate-300" className="p-3 rounded mt-2 bg-slate-50 border placeholder-slate-300"
placeholder="ex: code 'n chill, sales, birthdays, sprint 7..." placeholder="ex: presentation review process..."
value={tldr} value={tldr}
onChange={(e) => setTldr(e.target.value)} onChange={(e) => setTldr(e.target.value)}
/> />