Fix for toneMapping mode check (#537)

greggman · web-flow · commit 58fd7e1b0e36 · 2025-09-02T11:29:52.000-07:00
The spec says `toneMapping` won't exist if this isn't supported.
diff --git a/sample/particles/main.ts b/sample/particles/main.ts
@@ -348,7 +348,7 @@ function getHdrFolderName() {
   }
   if (
     simulationParams.toneMappingMode === 'extended' &&
-    context.getConfiguration().toneMapping.mode !== 'extended'
+    context.getConfiguration().toneMapping?.mode !== 'extended'
   ) {
     return "HDR settings ⚠️ Browser doesn't support HDR canvas";
   }

Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,7 @@ function getHdrFolderName() {`
`348`	`348`	`}`
`349`	`349`	`if (`
`350`	`350`	`simulationParams.toneMappingMode === 'extended' &&`
`351`		`- context.getConfiguration().toneMapping.mode !== 'extended'`
	`351`	`+ context.getConfiguration().toneMapping?.mode !== 'extended'`
`352`	`352`	`) {`
`353`	`353`	`return "HDR settings ⚠️ Browser doesn't support HDR canvas";`
`354`	`354`	`}`