Add text wrapping

CS3219-AY2425S1 · Nov 13, 2024 · 49e6c5c · 49e6c5c
1 parent 68a9e34
commit 49e6c5c
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/peerprep-fe/src/app/(main)/components/filter/TopicsPopover.tsx b/peerprep-fe/src/app/(main)/components/filter/TopicsPopover.tsx
@@ -48,18 +48,16 @@ export function TopicsPopover({
 
   const handleTopicSelection = (selectedTopic: string) => {
     if (multiselect) {
-      // Multiple selection mode
       const newSelectedTopics = selectedTopics.includes(selectedTopic)
         ? selectedTopics.filter((t) => t !== selectedTopic)
         : [...selectedTopics, selectedTopic];
       onChange(newSelectedTopics);
     } else {
-      // Single selection mode
       const newSelectedTopics = selectedTopics.includes(selectedTopic)
-        ? [] // Deselect if clicking the same topic
-        : [selectedTopic]; // Select only the clicked topic
+        ? []
+        : [selectedTopic];
       onChange(newSelectedTopics);
-      setOpen(false); // Close popover after selection in single select mode
+      setOpen(false);
     }
   };
 
@@ -116,18 +114,18 @@ export function TopicsPopover({
                 <Button
                   key={topic}
                   variant="ghost"
-                  className="justify-start"
+                  className="h-auto min-h-[2.5rem] justify-start whitespace-normal text-left"
                   onClick={() => handleTopicSelection(topic)}
                 >
                   <Check
                     className={cn(
-                      'mr-2 h-4 w-4',
+                      'mr-2 h-4 w-4 shrink-0',
                       selectedTopics.includes(topic)
                         ? 'opacity-100'
                         : 'opacity-0',
                     )}
                   />
-                  {topic}
+                  <span className="break-words">{topic}</span>
                 </Button>
               ))}
             </div>