Merge pull request #640 from nextcloud/fix/neon/theme

fix(neon): override equality operators in NeonTheme

packages/neon/neon/lib/src/theme/branding.dart

@@ -58,9 +58,7 @@ class Branding {
     if (identical(this, other)) {
       return true;
     }
-    if (other.runtimeType != runtimeType) {
-      return false;
-    }
+
     return other is Branding && name == other.name && logo == other.logo && legalese == other.legalese;
   }
 }

packages/neon/neon/lib/src/theme/color_scheme.dart

@@ -50,9 +50,7 @@ class NeonColorScheme {
     if (identical(this, other)) {
       return true;
     }
-    if (other.runtimeType != runtimeType) {
-      return false;
-    }
+
     return other is NeonColorScheme && other.primary == primary;
   }
 }

packages/neon/neon/lib/src/theme/dialog.dart

@@ -56,9 +56,7 @@ class NeonDialogTheme {
     if (identical(this, other)) {
       return true;
     }
-    if (other.runtimeType != runtimeType) {
-      return false;
-    }
+
     return other is NeonDialogTheme && other.constraints == constraints;
   }
 }

packages/neon/neon/lib/src/theme/neon.dart

@@ -55,4 +55,23 @@ class NeonTheme extends ThemeExtension<NeonTheme> {
       dialogTheme: NeonDialogTheme.lerp(dialogTheme, other.dialogTheme, t),
     );
   }
+
+  @override
+  int get hashCode => Object.hashAll([
+        branding,
+        colorScheme,
+        dialogTheme,
+      ]);
+
+  @override
+  bool operator ==(final Object other) {
+    if (identical(this, other)) {
+      return true;
+    }
+
+    return other is NeonTheme &&
+        other.branding == branding &&
+        other.colorScheme == colorScheme &&
+        other.dialogTheme == dialogTheme;
+  }
 }