fix: limit jobsPerPage to max length of 2 chars + use Math.min

felixmosh · Jul 26, 2023 · af96478 · af96478
1 parent c3435dd
commit af96478
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/ui/src/components/SettingsModal/SettingsModal.tsx b/packages/ui/src/components/SettingsModal/SettingsModal.tsx
@@ -16,8 +16,8 @@ const pollingIntervals = [-1, 3, 5, 10, 20, 60, 60 * 5, 60 * 15].map((interval)
     interval < 0
       ? 'Off'
       : Math.floor(interval / 60) === 0
-        ? `${interval} seconds`
-        : `${interval / 60} minutes`,
+      ? `${interval} seconds`
+      : `${interval / 60} minutes`,
   value: `${interval}`,
 }));
 
@@ -49,9 +49,10 @@ export const SettingsModal = ({ open, onClose }: SettingsModalProps) => {
         type="number"
         min="1"
         max="50"
+        maxLength={2}
         onChange={(event) => {
           const jobsPerPage = +event.target.value;
-          setSettings({ jobsPerPage: jobsPerPage > 50 ? 50 : jobsPerPage });
+          setSettings({ jobsPerPage: Math.min(jobsPerPage, 50) });
         }}
       />
       <SwitchField