Skip to content

Commit

Permalink
ui: fixes issue with location push/replace
Browse files Browse the repository at this point in the history
Signed-off-by: pamfilos <[email protected]>
  • Loading branch information
pamfilos committed Apr 5, 2024
1 parent 353e7c1 commit 9cbc023
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion ui/src/components/search/CheckboxFacet.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ const CheckboxFacet: React.FC<CheckboxFacetProps> = ({
}

setFilters(updated_filters);
router.replace(pathname + '?' + createQueryString(type, updated_filters))
router.push(pathname + '?' + createQueryString(type, updated_filters))
};

return (
Expand Down
2 changes: 1 addition & 1 deletion ui/src/components/search/SearchPagination.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ const SearchPagination: React.FC<SearchPagination> = ({ count, params }) => {
const params = new URLSearchParams(searchParams)
params.set("page", `${page}`);

router.replace(pathname + (params.toString() ? `?${params.toString()}` : ''));
router.push(pathname + (params.toString() ? `?${params.toString()}` : ''));
};

return (
Expand Down
7 changes: 4 additions & 3 deletions ui/src/components/search/YearFacet.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -65,12 +65,12 @@ const YearFacet = ({ data }: any) => {
const onSliderAfterChange = (data: number[]) => {
setSliderEndpoints(data)
const params = createQueryString('publication_year__range', [{x:data[0]}, {x:data[1]}]);
router.replace(pathname + (params ? `?${params.toString()}` : ""))
router.push(pathname + (params ? `?${params.toString()}` : ""))
};

const onBarClick = (value: YearFacetData) => {
const params = createQueryString('publication_year__range', [value, value]);
router.replace(pathname + (params ? `?${params.toString()}` : ""))
router.push(pathname + (params ? `?${params.toString()}` : ""))
};

const onBarMouseHover = (bar: YearFacetData) => {
Expand All @@ -83,7 +83,7 @@ const YearFacet = ({ data }: any) => {
const params = new URLSearchParams(searchParams);
params.delete('publication_year__range');
params.delete('page');
router.replace(pathname + (params.toString() ? `?${params.toString()}` : ""))
router.push(pathname + (params.toString() ? `?${params.toString()}` : ""))
};

let marks: any = {}
Expand Down Expand Up @@ -130,6 +130,7 @@ const YearFacet = ({ data }: any) => {
</div>
<Slider
range
disabled={Object.keys(marks).length <= 1 }
step={null}
className="year-facet-slider"
onChange={onSliderChange}
Expand Down

0 comments on commit 9cbc023

Please sign in to comment.