Merge pull request #991 from aaronreed708/fix-mintarget-area

#990: fix minimum target area broken
finos · Aug 9, 2024 · 8b1c6fb · 8b1c6fb
2 parents 7f569ef + 7c5e0f6
commit 8b1c6fb
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/code/src/ui/src/pages/atoms/MinimumTargetAtom.tsx b/code/src/ui/src/pages/atoms/MinimumTargetAtom.tsx
@@ -44,8 +44,8 @@ export const MinimumTargetAtom: React.FC<Props> = ({ atoms }) => {
         }
         selectables = selectables.sort();
         if (!selectables) return;
-        for (var j = 0; j < selectables.length; i++) {
-            const s = selectables[i].toString() + 'px';
+        for (var j = 0; j < selectables.length; j++) {
+            const s = selectables[j].toString() + 'px';
             r.push(
                 <MenuItem key={s} value={selectables[j]}>
                     {' '}
@@ -71,16 +71,16 @@ export const MinimumTargetAtom: React.FC<Props> = ({ atoms }) => {
     const renderMobileMinTargetSelectables = () => {
         var r = [];
         var selectables = [44, 48];
-        for (var i = 3; i <= 7; i++) {
-            const newVal = (atoms.gridSettings.grid.getValue() || 8) * i;
+        for (var k = 3; k <= 7; k++) {
+            const newVal = (atoms.gridSettings.grid.getValue() || 8) * k;
             if (newVal !== 44 && newVal !== 48) selectables.push(newVal);
         }
         selectables = selectables.sort();
         if (!selectables) return;
-        for (var k = 0; k < selectables.length; k++) {
-            const s = selectables[i].toString() + 'px';
+        for (var l = 0; l < selectables.length; l++) {
+            const s = selectables[l].toString() + 'px';
             r.push(
-                <MenuItem key={s} value={selectables[k]}>
+                <MenuItem key={s} value={selectables[l]}>
                     {' '}
                     {s}{' '}
                 </MenuItem>