fix(fs.shared): Linter warnings

packages/freesewing.shared/components/workbench/input-measurement.js

@@ -12,7 +12,7 @@ const MeasurementInput = ({ m, gist, app, updateMeasurements }) => {
   const prefix = (app.site === 'org') ? '' : 'https://freesewing.org'
   const title = app.t(`measurements.${m}`)
   const isValid = input => {
-    if (input === null || input === '') return null
+    if (input === '') return ''
     return !isNaN(input)
   }
 
@@ -26,9 +26,9 @@ const MeasurementInput = ({ m, gist, app, updateMeasurements }) => {
     } else setValid(false)
   }
 
-  const [val, setVal] = useState(gist?.measurements?.[m] || null)
+  const [val, setVal] = useState(gist?.measurements?.[m] || '')
   const [valid, setValid] = useState(typeof gist?.measurements?.[m] === 'undefined'
-    ? null :
+    ? '' :
     isValid(gist.measurements[m])
   )
 
@@ -68,14 +68,14 @@ const MeasurementInput = ({ m, gist, app, updateMeasurements }) => {
         <span className={`
           ${valid === false && 'bg-error text-neutral-content'}
           ${valid === true && 'bg-success text-neutral-content'}
-          ${valid === null && 'bg-base-200 text-base-content'}
+          ${valid === '' && 'bg-base-200 text-base-content'}
        `}>
           cm
         </span>
       </label>
       <label className="label">
         <span className="label-text-alt">
-          {valid === null
+          {valid === ''
             ? ''
             : valid
             ? 'Looks good'

packages/freesewing.shared/components/workbench/measurements.js

@@ -1,3 +1,4 @@
+import React from 'react'
 import MeasurementInput from './input-measurement.js'
 import { withBreasts, withoutBreasts } from 'pkgs/models/src/index.js'
 import nonHuman from './non-human-measurements.js'
@@ -52,11 +53,11 @@ const WorkbenchMeasurements = ({ app, pattern, gist, updateGist }) => {
         <summary><h2 className="inline-block">{app.t('cfp.preloadMeasurements')}</h2></summary>
         <div className="ml-2 pl-4 border-l-2">
           {Object.keys(groups).map(group => (
-            <details>
+            <details key={group}>
               <summary><h3 className="inline-block">{app.t(`app.${group}`)}</h3></summary>
               <div className="ml-2 pl-4 border-l-2">
                 {Object.keys(icons).map(type => (
-                  <>
+                  <React.Fragment key={type}>
                     <h4>{app.t(`app.${type}Breasts`)}</h4>
                     <ul className="flex flex-row flex-wrap gap-2">
                       {Object.keys(groups[group][type]).map((m) => (
@@ -75,7 +76,7 @@ const WorkbenchMeasurements = ({ app, pattern, gist, updateGist }) => {
                         </li>
                       ))}
                     </ul>
-                  </>
+                  </React.Fragment>
                 ))}
               </div>
             </details>

packages/freesewing.shared/components/workbench/menu.js

@@ -23,7 +23,7 @@ const ModeButtons = props => props.children.length === 0
               ? 'text-secondary border-secondary sm:text-secondary-focus sm:border-secondary-focus'
               : 'text-base-content sm:text-neutral-content'
             }
-          `}>
+          `} onClick={mode.onClick}>
             <span className={`
               text-3xl mr-2 inline-block p-0 leading-3
               ${mode.name === props.mode
@@ -88,7 +88,7 @@ const WorkbenchMenu = props => {
   return (
     <nav className="smmax-w-96 grow mb-12">
       {Object.keys(groups).map(group => (
-        <details className='py-1'>
+        <details className='py-1' key={group}>
           <summary className={`
             flex flex-row uppercase gap-4 font-bold text-lg
             hover:cursor-row-resize

packages/freesewing.shared/components/wrappers/workbench.js

@@ -21,7 +21,7 @@ const defaultGist = (pattern, language='en') => ({
 
 const hasRequiredMeasurements = (pattern, gist) => {
   for (const m of pattern.config.measurements) {
-    console.log(m)
+    //console.log(m)
   }
 }
 
@@ -75,6 +75,8 @@ const WorkbenchWrapper = ({ app, pattern }) => {
           updateGist={updateGist}
         />
       )}
+      <pre>{JSON.stringify(mode, null, 2)}</pre>
+      <pre>{JSON.stringify(gist, null, 2)}</pre>
     </Layout>
   )
 }