Imbus/TTime
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix week validation in NewWeeklyReport component for safari and firefox

Browse source
This commit is contained in:
Davenludd 2024-03-20 15:32:14 +01:00
parent 847073c6f8
commit caa793d036
1 changed files with 3 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
frontend/src/Components/NewWeeklyReport.tsx
View file

@ -45,8 +45,8 @@ export default function NewWeeklyReport(): JSX.Element {
<div className="border-4 border-black bg-white flex flex-col justify-start min-h-[65vh] h-fit w-[50vw] rounded-3xl overflow-scroll space-y-[2vh] p-[30px] items-center"> <div className="border-4 border-black bg-white flex flex-col justify-start min-h-[65vh] h-fit w-[50vw] rounded-3xl overflow-scroll space-y-[2vh] p-[30px] items-center">
<form <form
onSubmit={(e) => { onSubmit={(e) => {
if (week === 0) { if (week === 0 || week > 53 || week < 1) {
alert("Please enter a week number"); alert("Please enter a valid week number");
e.preventDefault(); e.preventDefault();
return; return;
} }
@ -81,12 +81,7 @@ export default function NewWeeklyReport(): JSX.Element {
placeholder="Week (Numbers Only)" placeholder="Week (Numbers Only)"
onChange={(e) => { onChange={(e) => {
const weekNumber = parseInt(e.target.value); const weekNumber = parseInt(e.target.value);
if (isNaN(weekNumber) || weekNumber < 1 || weekNumber > 53) {
setWeek(0);
return;
} else {
setWeek(weekNumber); setWeek(weekNumber);
}
}} }}
onKeyDown={(event) => { onKeyDown={(event) => {
const keyValue = event.key; const keyValue = event.key;