Skip to content

feat: make workspace search bar remember text #9759

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Sep 20, 2023
Merged

feat: make workspace search bar remember text #9759

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
2f92f86
minor: Add useEffectEvent polyfill
Parkreiner Sep 18, 2023
e2d3a67
chore: update filter to have better callback support
Parkreiner Sep 18, 2023
eeb3083
docs: Clean up comments
Parkreiner Sep 18, 2023
553ce89
fix: add localStorage to useWorkspacesFilter
Parkreiner Sep 18, 2023
b06d772
refactor: Centralize stable useSearchParams
Parkreiner Sep 18, 2023
df1f8df
refactor: clean up filter to be fully pure on mount
Parkreiner Sep 18, 2023
29656c1
chore: add tests for useEffectEvent
Parkreiner Sep 19, 2023
54a5940
wip: commit progress for searchbar fix
Parkreiner Sep 19, 2023
ae5b359
chore: clean up WorkspacesPage
Parkreiner Sep 19, 2023
7dd1b4f
fix: add logic for syncing queries with search params
Parkreiner Sep 19, 2023
ab9de0c
chore: Rename initialValue to fallbackFilter
Parkreiner Sep 19, 2023
391648a
chore: Remove todo comment
Parkreiner Sep 19, 2023
7e15671
Merge branch 'main' into workspace-searchbar-storage
Parkreiner Sep 19, 2023
60e9b10
Merge branch 'main' into workspace-searchbar-storage
Parkreiner Sep 20, 2023
dfad46f
refactor: update code to use useEffectEvent
Parkreiner Sep 20, 2023
dcd8691
docs: clean up comments for clarity
Parkreiner Sep 20, 2023
3a7efd5
fix: update url check to use regex
Parkreiner Sep 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: update code to use useEffectEvent
  • Loading branch information
@Parkreiner
Parkreiner committed Sep 20, 2023
commit dfad46f54c4151239798d29a62567a1c73dc738b
16 changes: 6 additions & 10 deletions site/src/components/Filter/filter.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ import Divider from "@mui/material/Divider";
import OpenInNewOutlined from "@mui/icons-material/OpenInNewOutlined";

import { useDebouncedFunction } from "hooks/debounce";
import { useEffectEvent } from "hooks/hookPolyfills";

export type PresetFilter = {
name: string;
Expand Down Expand Up @@ -55,18 +56,13 @@ export const useFilter = ({
const [searchParams, setSearchParams] = searchParamsResult;
const query = searchParams.get(useFilterParamsKey) ?? fallbackFilter;

// Using ref approach to be defensive against useSearchParams not returning a
// stable set function on each render. Effect must run before query effect
const setSearchParamsRef = useRef(setSearchParams);
useEffect(() => {
setSearchParamsRef.current = setSearchParams;
}, [setSearchParams]);
// Stabilizing reference to setSearchParams from one central spot, just to be
// on the extra careful side. Don't want effects over-running
const stableSetSearchParams = useEffectEvent(setSearchParams);

// Keep params synced with query, even as query changes from outside sources
useEffect(() => {
const setSearchParams = setSearchParamsRef.current;

setSearchParams((currentParams) => {
stableSetSearchParams((currentParams) => {
const currentQuery = currentParams.get(useFilterParamsKey);

if (query === "") {
Expand All @@ -77,7 +73,7 @@ export const useFilter = ({

return currentParams;
});
}, [query]);
}, [stableSetSearchParams, query]);

const update = (newValues: string | FilterValues) => {
const serialized =
Expand Down