feat(Topics): add sort URL param to topic pages.

This will track the sort option on topic pages which can be either "Relevance" or "Chronological".
Sefaria · Sep 6, 2023 · b4a5645 · b4a5645
1 parent 7602cfa
commit b4a5645
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/static/js/Misc.jsx b/static/js/Misc.jsx
@@ -364,11 +364,12 @@ const FilterableList = ({
   filterFunc, sortFunc, renderItem, sortOptions, getData, data, renderEmptyList,
   renderHeader, renderFooter, showFilterHeader, refreshData, initialFilter,
   scrollableElement, pageSize, onDisplayedDataChange, initialRenderSize,
-  bottomMargin, containerClass, onSetSort, initialSort,
+  bottomMargin, containerClass, onSetSort, externalSortOption,
 }) => {
   const [filter, setFilter] = useState(initialFilter || '');
-  const [sortOption, setSortOption] = useState(initialSort || sortOptions[0]);
+  const [internalSortOption, setSortOption] = useState(externalSortOption || sortOptions[0]);
   const [displaySort, setDisplaySort] = useState(false);
+  const sortOption = externalSortOption || internalSortOption;
 
   // Apply filter and sort to the raw data
   const processData = rawData => rawData ? rawData

diff --git a/static/js/TopicPage.jsx b/static/js/TopicPage.jsx
@@ -572,7 +572,7 @@ const TopicPageTab = ({
             onDisplayedDataChange={onDisplayedDataChange}
             initialRenderSize={initialRenderSize}
             onSetSort={onSetTopicSort}
-            initialSort={topicSort}
+            externalSortOption={topicSort}
             data={data}
           />
         </div> : <LoadingMessage />