207 buoy fields

[IanMayo]

Fixes #207

[netlify]

✅ Deploy Preview for reactol ready!

Name	Link
🔨 Latest commit	fc2dec0
🔍 Latest deploy log	https://app.netlify.com/sites/reactol/deploys/679c99662fd1e20008c6f840
😎 Deploy Preview	https://deploy-preview-214--reactol.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[Copilot]

Copilot reviewed 5 out of 11 changed files in this pull request and generated 2 comments.

Files not reviewed (6)

• src/components/BuoyFieldForm/index.css: Language not supported
• src/components/spatial/BuoyField/index.css: Language not supported
• src/mock/buoyfield1.json: Language not supported
• src/constants.ts: Evaluated as low risk
• src/components/spatial/Map/index.tsx: Evaluated as low risk
• src/components/Properties/index.tsx: Evaluated as low risk

Comments suppressed due to low confidence (2)

src/components/BuoyFieldForm/index.tsx:43

• The convert function does not handle cases where time or timeEnd is not provided, which could lead to undefined values being assigned to dTime and dTimeEnd. Consider adding checks to handle these cases.

const convert = (shape: Readonly<BuoyFieldProps>): FormTypeProps => {

src/data/buoyfield1.ts:9

• [nitpick] The color '#FF0' should be represented as '#FFFF00' for consistency with other color definitions.

color: '#FF0',

[Copilot]

The key prop should be more stable. Consider using a unique identifier from the feature properties or coordinates.

Suggested change

	{ isVisible && isSelected && locations && locations.length > 0 && locations.map((location: [number, number], index: number) => <CircleMarker key={'shiny-' + index + '-' + color} radius={8}
	{ isVisible && isSelected && locations && locations.length > 0 && locations.map((location: [number, number]) => <CircleMarker key={'shiny-' + feature.id + '-' + location.join(',')} radius={8}

[Copilot]

The eventHandlers object should be memoized to avoid unnecessary re-renders. Use useMemo or useCallback.