fix: document customizer requirement

100mslive · Jan 3, 2024 · 53e20eb · 53e20eb
1 parent d62606d
commit 53e20eb
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/packages/roomkit-react/src/Prebuilt/provider/roomLayoutProvider/index.tsx b/packages/roomkit-react/src/Prebuilt/provider/roomLayoutProvider/index.tsx
@@ -18,11 +18,14 @@ export const RoomLayoutContext = React.createContext<
   | undefined
 >(undefined);
 
+// The default merge in lodash merges the values of current layout and the changes.
+// This behaviour makes changes in array based values inconsistent since a union happens.
+// The customizer uses the new value provided if one of the values is an array
 function customizer(objValue: any, srcValue: any) {
-  if (isArray(objValue) && isArray(srcValue)) {
+  if (isArray(objValue) || isArray(srcValue)) {
     return srcValue;
   }
-  // default mergeWith behaviour is followed
+  // default merge behaviour is followed
   return undefined;
 }