Eeeven more lint

Author: jkluge

my-app/src/presenters/ListViewPresenter.jsx

@@ -7,6 +7,14 @@ import PrerequisitePresenter from './PrerequisitePresenter.jsx';
 
 const ListViewPresenter = observer(({ model }) => {
 
+    const handleFavouriteClick = (course) => {
+        if (model.favourites.some(fav => fav.code === course.code)) {
+            model.removeFavourite(course);
+        } else {
+            model.addFavourite(course);
+        }
+    };
+
     const [isPopupOpen, setIsPopupOpen] = useState(false);
     const [selectedCourse, setSelectedCourse] = useState(null);
     const preP = <PrerequisitePresenter model={model} selectedCourse={selectedCourse}/>
@@ -19,14 +27,6 @@ const ListViewPresenter = observer(({ model }) => {
         model.removeFavourite(course);
     }
 
-    function handleFavouriteClick(course){
-        if (props.favouriteCourses.some(fav => fav.code === course.code)) {
-            props.removeFavourite(course);
-        } else {
-            props.addFavourite(course);
-        }
-    };
-
     return <ListView
         courses={model.courses}
         searchResults={model.currentSearch}
@@ -37,6 +37,7 @@ const ListViewPresenter = observer(({ model }) => {
         setIsPopupOpen={setIsPopupOpen}
         setSelectedCourse={setSelectedCourse}
         popUp={popup}
+        handleFavouriteClick={handleFavouriteClick}
 
     />;
 });

my-app/src/views/ListView.jsx

@@ -11,18 +11,12 @@ function ListView(props) {
             { ...prevState, [courseCode]: !prevState[courseCode] }
         ));
     };
-  
 
     return (
         <div className="relative bg-white text-black p-2 flex flex-col gap-5 h-full overflow-auto">
             {coursesToDisplay.length > 0 ? (
                 coursesToDisplay.map((course) => (
                     <div
-                    onClick={() => {
-                        console.log('Clicked:', course); // check browser console
-                        props.setSelectedCourse(course);
-                        props.setIsPopupOpen(true);
-                    }}
                         key={course.code}
                         className="p-5 hover:bg-blue-100 flex items-center border border-b-black border-solid w-full rounded-lg cursor-pointer"
                     >
@@ -40,10 +34,7 @@ function ListView(props) {
                             {course.description.length > 150 && (
                                 <span
                                     className="text-blue-500 cursor-pointer"
-                                    onClick={(e) => {
-                                        e.stopPropagation();  // Prevent the event from bubbling up.
-                                        toggleReadMore(course.code);
-                                    }}
+                                    onClick={() => toggleReadMore(course.code)}
                                 >
                                     {readMoreState[course.code]
                                         ?
@@ -55,10 +46,7 @@ function ListView(props) {
                             <div>
                                 <button
                                     className="text-yellow-500 cursor-pointer"
-                                    onClick={(e) => {
-                                        e.stopPropagation();  // Prevent the event from bubbling up.
-                                        handleFavouriteClick(course);
-                                    }}
+                                    onClick={() => props.handleFavouriteClick(course)}
                                 >
                                     {props.favouriteCourses.some(fav => fav.code === course.code)
                                         ?
@@ -78,9 +66,8 @@ function ListView(props) {
                     color="#000061"
                 />
             )}
-             {props.popUp}
         </div>
     );
 }
 
-export default ListView;
+export default ListView;