fixed a bug where scrollToMessage

Author: danney-chun

src/modules/App/DesktopLayout.tsx

@@ -84,7 +84,7 @@ export const DesktopLayout: React.FC<DesktopLayoutProps> = (props: DesktopLayout
     onSearchClick: () => {
       setShowSettings(false);
       setShowThread(false);
-      setShowSearch(!showSearch);
+      setShowSearch((prev: boolean) => { return !prev; });
     },
     onReplyInThread: onClickThreadReply,
     onQuoteMessageClick: ({ message }) => {

src/modules/App/types.ts

@@ -43,7 +43,7 @@ export interface DesktopLayoutProps extends AppLayoutProps, SubLayoutCommonProps
   showSettings: boolean;
   setShowSettings: React.Dispatch<boolean>;
   showSearch: boolean;
-  setShowSearch: React.Dispatch<boolean>;
+  setShowSearch: React.Dispatch<React.SetStateAction<boolean>>;
   // thread
   showThread: boolean;
   setShowThread: React.Dispatch<boolean>;

src/modules/GroupChannel/context/hooks/useGroupChannel.ts

@@ -126,21 +126,21 @@ export const useGroupChannel = () => {
 
     if (message) {
       const topOffset = getMessageTopOffset(message.createdAt);
-      if (topOffset) state.scrollPubSub.publish('scroll', { top: topOffset, animated: scrollAnimated });
+      if (topOffset !== null) state.scrollPubSub.publish('scroll', { top: topOffset, animated: scrollAnimated });
       if (messageFocusAnimated ?? true) setAnimatedMessageId(messageId);
     } else if (state.initialized) {
       await state.resetWithStartingPoint(createdAt);
       setTimeout(() => {
         const topOffset = getMessageTopOffset(createdAt);
-        if (topOffset) {
+        if (topOffset !== null) {
           state.scrollPubSub.publish('scroll', {
             top: topOffset,
             lazy: false,
             animated: scrollAnimated,
           });
         }
         if (messageFocusAnimated ?? true) setAnimatedMessageId(messageId);
-      });
+      }, 500);
     }
     clickHandler.activate();
   }, [

src/modules/MessageSearch/context/hooks/useMessageSearch.ts

@@ -60,7 +60,7 @@ const useMessageSearch = () => {
     store.setState(state => ({ ...state, allMessages: [] }));
   }, [store]);
 
-  const setSelectedMessageId = (messageId: number) => useCallback(() => {
+  const setSelectedMessageId = useCallback((messageId: number) => {
     store.setState(state => ({ ...state, selectedMessageId: messageId }));
   }, [store]);