Fix deep-equality for array properties (#4672)

.changeset/merge-nodes-issue

@@ -1,5 +0,0 @@
----
-"slate": patch
----
-
-Do not remove empty node in merge operation if it is first children in its parent.

.changeset/wicked-mayflies-approve.md

@@ -0,0 +1,5 @@
+---
+'slate': minor
+---
+
+Fix - deep-equals was always returning true when array props were equals.

packages/slate/src/utils/deep-equal.ts

@@ -24,7 +24,6 @@ export const isDeepEqual = (
       for (let i = 0; i < a.length; i++) {
         if (a[i] !== b[i]) return false
       }
-      return true
     } else if (a !== b) {
       return false
     }

packages/slate/test/utils/deep-equal/deep-equals-with-array.js

@@ -0,0 +1,20 @@
+import { isDeepEqual } from '../../../src/utils/deep-equal'
+
+export const input = {
+  objectA: {
+    text: 'same text',
+    array: ['array-content'],
+    bold: true,
+  },
+  objectB: {
+    text: 'same text',
+    array: ['array-content'],
+    bold: true,
+  },
+}
+
+export const test = ({ objectA, objectB }) => {
+  return isDeepEqual(objectA, objectB)
+}
+
+export const output = true

packages/slate/test/utils/deep-equal/deep-not-equals-with-array.js

@@ -0,0 +1,20 @@
+import { isDeepEqual } from '../../../src/utils/deep-equal'
+
+export const input = {
+  objectA: {
+    text: 'same text',
+    array: ['array-content'],
+    bold: true,
+  },
+  objectB: {
+    text: 'same text',
+    array: ['array-content'],
+    bold: false,
+  },
+}
+
+export const test = ({ objectA, objectB }) => {
+  return isDeepEqual(objectA, objectB)
+}
+
+export const output = false