From cbc4f4a7c5d85acaf15d5aadc22677409b4a35f7 Mon Sep 17 00:00:00 2001
From: Jonas de Luna Skulberg <jonasdls@gmail.com>
Date: Tue, 10 Sep 2024 21:00:30 +0200
Subject: [PATCH] Update to use dropdown

---
 app/components/Comments/CommentView.css      |  17 +--
 app/components/Comments/CommentView.tsx      | 127 ++++++++-----------
 app/routes/forum/components/ThreadDetail.tsx |   1 -
 3 files changed, 60 insertions(+), 85 deletions(-)

diff --git a/app/components/Comments/CommentView.css b/app/components/Comments/CommentView.css
index 535f029b28..009ba9b4e1 100644
--- a/app/components/Comments/CommentView.css
+++ b/app/components/Comments/CommentView.css
@@ -1,16 +1,9 @@
 .headerContainer {
-  margin-bottom: 0.25rem;
+  margin-bottom: var(--spacing-sm);
 }
 
-.sortIcon {
-  flex-direction: column;
-}
-
-.iconWrapper {
-  margin-left: 0.25rem;
-}
-
-.sortList {
-  margin-left: 0.5rem;
-  min-width: 10rem;
+.dropdownElement {
+  padding: var(--spacing-sm);
+  cursor: pointer;
+  width: var(--spacing-ss);
 }
diff --git a/app/components/Comments/CommentView.tsx b/app/components/Comments/CommentView.tsx
index be40613005..7bb26b1bd8 100644
--- a/app/components/Comments/CommentView.tsx
+++ b/app/components/Comments/CommentView.tsx
@@ -1,15 +1,10 @@
-import {
-  FilterSection,
-  Flex,
-  Icon,
-  LoadingIndicator,
-} from '@webkom/lego-bricks';
-import { ListFilter } from 'lucide-react';
+import { Flex, Icon, LoadingIndicator } from '@webkom/lego-bricks';
+import { ArrowDownUpIcon } from 'lucide-react';
 import moment from 'moment';
-import { useEffect, useState, type CSSProperties } from 'react';
+import { useState, type CSSProperties } from 'react';
 import CommentForm from 'app/components/CommentForm';
+import Dropdown from 'app/components/Dropdown';
 import { generateTreeStructure } from 'app/utils';
-import { SelectInput } from '../Form';
 import CommentTree from './CommentTree';
 import styles from './CommentView.css';
 import type Comment from 'app/store/models/Comment';
@@ -36,13 +31,9 @@ const orderingOptions: Array<Option> = [
     value: 'createdAtInv',
   },
   {
-    label: 'Likes',
+    label: 'Reaksjoner',
     value: 'reactionsGrouped',
   },
-  {
-    label: 'Kontroversielle',
-    value: 'reactionsControversial',
-  },
 ];
 
 type Option = {
@@ -50,13 +41,6 @@ type Option = {
   value: string;
 };
 
-const getReactionScore = (comment: Comment, emojiString: string): number => {
-  const count =
-    comment.reactionsGrouped.find((reaction) => reaction.emoji === emojiString)
-      ?.count || 0;
-  return count;
-};
-
 const Title = ({ displayTitle }: { displayTitle: boolean }) =>
   displayTitle && <h3>Kommentarer</h3>;
 
@@ -68,35 +52,30 @@ const CommentView = (props: Props) => {
     style,
     displayTitle = true,
     contentAuthors,
+    newOnTop,
   } = props;
   const commentFormProps = {
     contentTarget,
   };
-  const [ordering, setOrdering] = useState<Option>(orderingOptions[0]);
-  const [sortedComments, setSortedComments] = useState<Comment[]>(comments);
+  const [ordering, setOrdering] = useState<Option>(
+    newOnTop ? orderingOptions[0] : orderingOptions[1],
+  );
   const [displaySorting, setDisplaySorting] = useState(false);
-  const tree = generateTreeStructure(sortedComments);
 
-  useEffect(() => {
-    const sorted = [...comments].sort((a: Comment, b: Comment) => {
-      if (ordering.value === 'createdAt') {
-        return moment(b.createdAt).valueOf() - moment(a.createdAt).valueOf();
-      } else if (ordering.value === 'createdAtInv') {
-        return moment(a.createdAt).valueOf() - moment(b.createdAt).valueOf();
-      } else if (ordering.value === 'reactionsGrouped') {
-        const scoreA =
-          getReactionScore(a, ':+1:') - getReactionScore(a, ':-1:');
-        const scoreB =
-          getReactionScore(b, ':+1:') - getReactionScore(b, ':-1:');
-        return scoreB - scoreA;
-      } else if (ordering.value === 'reactionsControversial') {
-        return getReactionScore(b, ':-1:') - getReactionScore(a, ':-1:');
-      }
-      return 0;
-    });
-
-    setSortedComments(sorted);
-  }, [ordering, comments]);
+  const sortedComments = comments.sort((a: Comment, b: Comment) => {
+    if (ordering.value === 'createdAt') {
+      return moment(b.createdAt).valueOf() - moment(a.createdAt).valueOf();
+    } else if (ordering.value === 'createdAtInv') {
+      return moment(a.createdAt).valueOf() - moment(b.createdAt).valueOf();
+    } else if (ordering.value === 'reactionsGrouped') {
+      return (
+        b.reactionsGrouped.reduce((acc, reaction) => acc + reaction.count, 0) -
+        a.reactionsGrouped.reduce((acc, reaction) => acc + reaction.count, 0)
+      );
+    }
+    return 0;
+  });
+  const tree = generateTreeStructure(sortedComments);
 
   return (
     <div style={style}>
@@ -107,39 +86,43 @@ const CommentView = (props: Props) => {
         className={styles.headerContainer}
       >
         <Title displayTitle={displayTitle} />
-        <Flex flex-row alignItems="center" className={styles.iconWrapper}>
-          <Icon
-            name="arrow-down-wide-narrow"
-            size={20}
-            onClick={() => {
-              setDisplaySorting(!displaySorting);
-            }}
-            className={styles.sortIcon}
-            iconNode={<ListFilter />}
-          />
-          {displaySorting && (
-            <div className={styles.sortList}>
-              <FilterSection title="">
-                <SelectInput
-                  name="sorting_selector"
-                  value={ordering}
-                  onChange={(selectedOption: Option) => {
-                    setOrdering(selectedOption);
-                  }}
-                  isClearable={false}
-                  options={orderingOptions}
-                />
-              </FilterSection>
-            </div>
-          )}
-        </Flex>
+
+        <Dropdown
+          show={displaySorting}
+          toggle={() => setDisplaySorting(!displaySorting)}
+          triggerComponent={
+            <Icon
+              size={20}
+              className={styles.sortIcon}
+              iconNode={<ArrowDownUpIcon />}
+            />
+          }
+        >
+          <Dropdown.List>
+            {orderingOptions.map((o: Option, index: number) => (
+              <>
+                {index !== 0 && <Dropdown.Divider />}
+                <Dropdown.ListItem key={o.value}>
+                  <button
+                    onClick={() => {
+                      setOrdering(o);
+                      setDisplaySorting(!displaySorting);
+                    }}
+                  >
+                    {o.label}
+                  </button>
+                </Dropdown.ListItem>
+              </>
+            ))}
+          </Dropdown.List>
+        </Dropdown>
       </Flex>
 
       <Flex column gap="var(--spacing-sm)">
         {!formDisabled && <CommentForm {...commentFormProps} />}
 
-        <LoadingIndicator loading={!sortedComments}>
-          {sortedComments && (
+        <LoadingIndicator loading={!comments}>
+          {comments && (
             <CommentTree
               comments={tree}
               commentFormProps={commentFormProps}
diff --git a/app/routes/forum/components/ThreadDetail.tsx b/app/routes/forum/components/ThreadDetail.tsx
index ae92216042..c3717d2b4d 100644
--- a/app/routes/forum/components/ThreadDetail.tsx
+++ b/app/routes/forum/components/ThreadDetail.tsx
@@ -73,7 +73,6 @@ const ThreadDetail = () => {
             contentTarget={thread.contentTarget}
             comments={comments}
             contentAuthors={thread.createdBy}
-            newOnTop
           />
         )}
       </Page>