From b76cdb712a08f07c532b593cdfd5a00b240d9d85 Mon Sep 17 00:00:00 2001
From: Mikael Olausson <104485047+mikaelol@users.noreply.github.com.>
Date: Tue, 25 Mar 2025 15:59:19 +0100
Subject: [PATCH] fix(ui): switch to sort by oldest when specifying timestamp

---
 .../containers/Topic/Topic/TopicData/TopicData.jsx | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/client/src/containers/Topic/Topic/TopicData/TopicData.jsx b/client/src/containers/Topic/Topic/TopicData/TopicData.jsx
index 625514969..cfee8a242 100644
--- a/client/src/containers/Topic/Topic/TopicData/TopicData.jsx
+++ b/client/src/containers/Topic/Topic/TopicData/TopicData.jsx
@@ -720,10 +720,14 @@ class TopicData extends Root {
       renderedOptions.push(
         <Dropdown.Item
           key={option}
-          onClick={() =>
-            this.setState({ sortBy: option }, () => {
-              this._searchMessages();
-            })
+          onClick={() => {
+              if (option === 'Newest' && this.state.endDatetime !== '') {
+                toast.warn('Sorting by newest with timestamp in large topics may not show data.');
+              }
+              this.setState({ sortBy: option }, () => {
+                this._searchMessages();
+              })
+            }
           }
         >
           <FontAwesomeIcon icon={faSortNumericDesc} aria-hidden={true} pull={'left'} /> {option}
@@ -1078,7 +1082,7 @@ class TopicData extends Root {
                           showTimeSelect
                           value={endDatetime}
                           onChange={value => {
-                            this.setState({ endDatetime: value }, () => {
+                            this.setState({ endDatetime: value, sortBy: "Oldest" }, () => {
                               this._searchMessages();
                             });
                           }}