Skip to content

Commit bb05ec0

Browse files
kexanakexana
committed
fixed upload
1 parent d077a32 commit bb05ec0

12 files changed

Lines changed: 1808 additions & 182 deletions

File tree

‎my-app/package-lock.json‎

Lines changed: 1751 additions & 115 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎my-app/package.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
"firebase": "^11.5.0",
2020
"fuse.js": "^7.1.0",
2121
"ldrs": "^1.1.6",
22+
"lint": "^0.8.19",
2223
"lodash.debounce": "^4.0.8",
2324
"lodash.throttle": "^4.1.1",
2425
"mobx": "^6.13.7",

‎my-app/src/model.js‎

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -215,11 +215,6 @@ export const model = {
215215
this.setFiltersChange();
216216
},
217217

218-
setApplyRemoveNullCourses() {
219-
this.filterOptions.applyRemoveNullCourses = !this.filterOptions.applyRemoveNullCourses;
220-
this.setFiltersChange();
221-
},
222-
223218
updateLevelFilter(level) {
224219
this.filterOptions.level = level;
225220
},
@@ -242,10 +237,6 @@ export const model = {
242237
this.filterOptions.eligibility = eligibility;
243238
},
244239

245-
updateDepartmentFilter(department) {
246-
this.filterOptions.department = department;
247-
},
248-
249240
updatePeriodFilter(period) {
250241
this.filterOptions.period = period;
251242
},

‎my-app/src/pages/App.jsx‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,11 @@ import { SearchbarPresenter } from '../presenters/SearchbarPresenter.jsx';
44
import { ListViewPresenter } from '../presenters/ListViewPresenter.jsx';
55
import { FilterPresenter } from "../presenters/FilterPresenter.jsx";
66
import { slide as Menu } from 'react-burger-menu';
7-
import { observer } from 'mobx-react-lite';
87

98

109

1110
function App({ model }) {
1211
const [sidebarIsOpen, setSidebarIsOpen] = useState(model.sidebarIsOpen);
13-
const toggleSidebar = () => {
14-
setSidebarIsOpen(!sidebarIsOpen);
15-
}
1612

1713
return (
1814
/* The sidebar styling(under the menu)*/

‎my-app/src/presenters/SidebarPresenter.jsx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const SidebarPresenter = observer(({ model }) => {
1616

1717
useEffect(() => {
1818
model.setFiltersChange();
19-
},[]);
19+
},[model]);
2020

2121
function handleLanguageFilterChange(param) {
2222
if (param === "English") {
@@ -219,9 +219,9 @@ const SidebarPresenter = observer(({ model }) => {
219219
const handleFileChange = (event) => {
220220
const truncatedCourses = model.courses.map(({ id, name }) => ({ id, name }));
221221
const file = event.target.files[0];
222-
//document.getElementById('PDF-Scraper-Error').style.visibility = "visible";
222+
document.getElementById('PDF-Scraper-Error').style.visibility = "visible";
223223
transcriptScraperFunction(file, setErrorMessage, setErrorVisibility, reApplyFilter, truncatedCourses);
224-
//document.getElementById('PDF-Scraper-Input').value = '';
224+
document.getElementById('PDF-Scraper-Input').value = '';
225225
setFileInputValue('');
226226

227227

‎my-app/src/views/Components/SideBarComponents/ButtonGroupField.jsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export default function ButtonGroupField(props) {
3131
};
3232
return (
3333
<div className="my-1">
34-
<div className="flex flex-col sm:inline-flex sm:flex-row rounded-lg shadow-2xs
34+
<div className="flex sm:inline-flex rounded-lg shadow-2xs
3535
w-full items-center font-medium text-white bg-[#aba8e0]">
3636
{props.items.map((item, index) => (
3737
<button

‎my-app/src/views/Components/SideBarComponents/CollapsibleCheckboxes.jsx‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import FilterEnableCheckbox from "./FilterEnableCheckbox";
33
import Tooltip from "./ToolTip";
44

55
const CollapsibleCheckboxes = (props) => {
6-
const [expandedLabel, setExpandedLabel] = useState(props?.initialValues?.map(i => i?.label));
76
const [expanded, setExpanded] = useState([]);
87
const [filterEnabled, setFilterEnabled] = useState(props.filterEnable);
98
const [checkedSubItems, setCheckedSubItems] = useState({});
@@ -83,7 +82,7 @@ const CollapsibleCheckboxes = (props) => {
8382
setInitalLoad(false);
8483
setExpanded(tempExpanded);
8584
}
86-
}, [rows]);
85+
}, [rows, initalLoad, props?.initialValues]);
8786

8887

8988

‎my-app/src/views/Components/SideBarComponents/CourseTranscriptList.jsx‎

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
1-
import { useState, useRef, forwardRef } from "react";
2-
import Tooltip from "./ToolTip";
1+
import { useState, forwardRef } from "react";
32

4-
export default function CourseTranscriptList(props) {
3+
const CourseTranscriptList = forwardRef((props,ref) => {
54
let local = [];
65
if (localStorage.getItem("completedCourses"))
76
local = JSON.parse(localStorage.getItem("completedCourses"));
@@ -31,14 +30,11 @@ export default function CourseTranscriptList(props) {
3130
localStorage.setItem("completedCourses", JSON.stringify(newitems));
3231
window.dispatchEvent(new Event("completedCourses changed"));
3332
props.reApplyFilter();
34-
if (props.checkboxRef && props.checkboxRef.current) {
35-
props.checkboxRef.current.click();
36-
}
33+
ref.current.click();
3734
};
3835

3936

4037
//=====================================
41-
const tooltipRef = useRef(null);
4238

4339
const tooltipClasses = [
4440
"absolute",
@@ -80,25 +76,25 @@ export default function CourseTranscriptList(props) {
8076
<div
8177
className="absolute inset-0 mt-30 pointer-events-none bg-gradient-to-b from-transparent to-[#553d65] z-100"
8278
></div>
83-
<div className="grid grid-cols-3 w-full max-[1200px]:grid-cols-2 max-[700px]:grid-cols-1 gap-1 sm:gap-2 overflow-y-auto overflow-x-hidden max-h-[180px] pb-10" style={{
79+
<div className="grid grid-cols-3 w-full max-[1200px]:grid-cols-2 gap-1 sm:gap-2 overflow-y-auto overflow-x-hidden max-h-[180px] pb-10" style={{
8480
scrollbarWidth: "thin",
8581
scrollbarColor: "#888 #f1f1f1",
8682
}}>
8783
{items.map((item, index) => (
8884
<div
8985
key={index}
90-
className="flex items-center bg-[#aba8e0] px-3 py-1 rounded-md shadow-md text-sm min-w-18 relative"
86+
className="flex items-center bg-[#aba8e0] px-3 py-1 rounded-md shadow-md text-sm min-w-18"
9187
>
9288
<div className="relative">
9389
<span className="flex-auto mr-2 peer">{item?.id}</span>
9490

9591
<div className={tooltipClasses}>
96-
{item?.name + (item?.is_in_DB ? "" : " (Course discontinued)")}
92+
{item?.name}
9793
</div>
9894
</div>
9995
<button
10096
onClick={() => removeItem(index)}
101-
className="text-violet-600 hover:text-red-700 font-bold text-sm hover:bg-red-300 rounded-md absolute right-2 top-1/2 -translate-y-1/2"
97+
className="text-violet-600 hover:text-red-700 font-bold text-sm hover:bg-red-300 rounded-md"
10298
>
10399
<svg
104100
xmlns="http://www.w3.org/2000/svg"
@@ -120,4 +116,6 @@ export default function CourseTranscriptList(props) {
120116
</div>
121117
</div>
122118
);
123-
}
119+
});
120+
121+
export default CourseTranscriptList;

‎my-app/src/views/Components/SideBarComponents/MultipleChoiceButtons.jsx‎

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export default function MultipleChoiceButtons(props) {
1111

1212
useEffect(() => {
1313
setFilterEnabled(props.filterEnable);
14-
})
14+
},[props.filterEnable])
1515

1616
const handleClick = (index) => {
1717
setSelectedItems((prev) => {
@@ -36,17 +36,18 @@ export default function MultipleChoiceButtons(props) {
3636
return "default-button-class"; // Fallback class for invalid index
3737
}
3838

39-
const baseClasses = `flex-auto py-1 px-4 inline-flex items-center gap-x-2 text-sm
40-
font-medium focus:z-10 border border-gray-200 shadow-2xs hover:bg-[#8785ac]
41-
focus:outline-hidden disabled:opacity-50 disabled:pointer-events-none pl-8`;
39+
const baseClasses = `flex-auto py-1 px-4 inline-flex items-center text-sm
40+
font-medium focus:z-10 border border-gray-200 shadow-2xs hover:bg-[#8785ac]
41+
focus:outline-hidden sm:pl-7 pl-4 text-center`;
4242
const activeClass = selectedItems[index] ? "bg-violet-500" : "bg-transparent";
4343
const roundedClasses =
4444
index === 0
45-
? "rounded-l-lg"
46-
: index === props.items.length - 1
47-
? "rounded-r-lg"
48-
: "border-l-0";
49-
return `${baseClasses} ${activeClass} ${roundedClasses}`;
45+
? "rounded-l-lg"
46+
: index === props.items.length - 1
47+
? "rounded-r-lg"
48+
: "border-l-0 ";
49+
const mobileClasses = "w-full sm:w-auto"; // Adjust for mobile
50+
return `${baseClasses} ${activeClass} ${roundedClasses} ${mobileClasses}`;
5051
};
5152

5253
return (
@@ -74,7 +75,7 @@ export default function MultipleChoiceButtons(props) {
7475
}}>
7576

7677
<div className="my-1">
77-
<div className="flex flex-col sm:inline-flex sm:flex-row rounded-lg shadow-2xs
78+
<div className="flex sm:inline-flex rounded-lg shadow-2xs
7879
w-full items-center font-medium text-white bg-[#aba8e0]">
7980
{props.items.map((item, index) => (
8081
<button

‎my-app/src/views/Components/SideBarComponents/SliderField.jsx‎

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,16 @@
1-
import React, { useState, useRef, useEffect } from "react";
1+
import React, { useState, useRef, useEffect, useMemo } from "react";
22
import FilterEnableCheckbox from "./FilterEnableCheckbox";
33
import Tooltip from "./ToolTip";
44

55
export default function UploadField(props) {
66
let paramFieldType = "slider";
77

8-
const values = [
8+
9+
const values = useMemo(() => [
910
1, 1.5, 2, 2.5, 3, 3.5, 4, 4.5, 5, 5.5,
1011
6, 7, 7.5, 8, 8.5, 9, 10, 11, 12, 13.5,
1112
14, 15, 20, 22.5, 30, 45
12-
];
13+
], []);
1314

1415
const [minIndex, setMinIndex] = useState(0);
1516
const [maxIndex, setMaxIndex] = useState(values.length - 1);
@@ -19,14 +20,14 @@ export default function UploadField(props) {
1920

2021
useEffect(() => {
2122
for (let i = 0; i < values.length; i++) {
22-
if (values[i] === props.initialValues[0]) {
23+
if (values[i] === props?.initialValues[0]) {
2324
setMinIndex(i);
2425
}
25-
if (values[i] === props.initialValues[1]) {
26+
if (values[i] === props?.initialValues[1]) {
2627
setMaxIndex(i);
2728
}
2829
}
29-
}, []); // Empty dependency array ensures this runs only once
30+
}, [props?.initialValues, values]); // Empty dependency array ensures this runs only once
3031

3132

3233
const handleDrag = (e, thumbType) => {
@@ -110,7 +111,7 @@ export default function UploadField(props) {
110111
window.addEventListener("mousemove", move);
111112
window.addEventListener("mouseup", up);
112113
}}
113-
onTouchStart={(e) => {
114+
onTouchStart={() => {
114115
const move = (ev) => handleDrag(ev, "min");
115116
const end = () => {
116117
window.removeEventListener("touchmove", move);
@@ -139,7 +140,7 @@ export default function UploadField(props) {
139140
window.addEventListener("mousemove", move);
140141
window.addEventListener("mouseup", up);
141142
}}
142-
onTouchStart={(e) => {
143+
onTouchStart={() => {
143144
const move = (ev) => handleDrag(ev, "max");
144145
const end = () => {
145146
window.removeEventListener("touchmove", move);

0 commit comments

Comments
 (0)