fix: lint issues

conceptadev · Nov 14, 2024 · 2de2a79 · 2de2a79
1 parent 753f8a8
commit 2de2a79
Show file tree

Hide file tree

Showing 9 changed files with 13 additions and 42 deletions.
diff --git a/packages/mix/lib/src/attributes/border/border_radius_dto.dart b/packages/mix/lib/src/attributes/border/border_radius_dto.dart
@@ -56,9 +56,6 @@ sealed class BorderRadiusGeometryDto<T extends BorderRadiusGeometry>
     properties.addUsingDefault('bottomStart', bottomStart);
     properties.addUsingDefault('bottomEnd', bottomEnd);
   }
-
-  @override
-  T resolve(MixData mix);
 }
 
 @MixableDto(generateUtility: false)

diff --git a/packages/mix/lib/src/attributes/border/shape_border_dto.dart b/packages/mix/lib/src/attributes/border/shape_border_dto.dart
@@ -38,9 +38,6 @@ sealed class ShapeBorderDto<T extends ShapeBorder> extends Dto<T> {
 
   @override
   ShapeBorderDto<T> merge(covariant ShapeBorderDto<T>? other);
-
-  @override
-  T resolve(MixData mix);
 }
 
 @immutable
@@ -86,9 +83,6 @@ abstract class OutlinedBorderDto<T extends OutlinedBorder>
 
   @override
   OutlinedBorderDto<T> merge(covariant OutlinedBorderDto<T>? other);
-
-  @override
-  T resolve(MixData mix);
 }
 
 @MixableDto()

diff --git a/packages/mix/lib/src/core/attribute.dart b/packages/mix/lib/src/core/attribute.dart
@@ -3,14 +3,12 @@ import 'package:flutter/widgets.dart';
 import '../internal/compare_mixin.dart';
 import 'dto.dart';
 
-abstract class Attribute with MergeableMixin, EqualityMixin {
+abstract class Attribute with MergeableMixin<Attribute>, EqualityMixin {
   const Attribute();
 
   // Used as the key to determine how
   // attributes get merged
   Object get mergeKey => runtimeType;
-  @override
-  Attribute merge(covariant Attribute? other);
 }
 
 /// Provides the ability to merge this object with another of the same type.

diff --git a/packages/mix/lib/src/core/dto.dart b/packages/mix/lib/src/core/dto.dart
@@ -6,15 +6,12 @@ import 'factory/mix_data.dart';
 import 'utility.dart';
 
 @immutable
-abstract class Dto<Value> with EqualityMixin, MergeableMixin {
+abstract class Dto<Value> with EqualityMixin, MergeableMixin<Dto> {
   const Dto();
 
   Value get defaultValue;
 
   Value resolve(MixData mix);
-  // /// Merges this object with [other], returning a new object of type [T].
-  @override
-  Dto merge(covariant Dto? other);
 }
 
 abstract class DtoUtility<Attr extends Attribute, D extends Dto<Value>, Value>

diff --git a/packages/mix/lib/src/core/modifier.dart b/packages/mix/lib/src/core/modifier.dart
@@ -2,7 +2,6 @@ import 'package:flutter/foundation.dart';
 import 'package:flutter/widgets.dart';
 
 import 'attribute.dart';
-import 'factory/mix_data.dart';
 import 'spec.dart';
 import 'utility.dart';
 
@@ -34,9 +33,6 @@ abstract class WidgetModifierSpecAttribute<
         Value extends WidgetModifierSpec<Value>> extends SpecAttribute<Value>
     with Diagnosticable {
   const WidgetModifierSpecAttribute();
-
-  @override
-  Value resolve(MixData mix);
 }
 
 abstract class WidgetModifierUtility<

diff --git a/packages/mix/lib/src/core/styled_widget.dart b/packages/mix/lib/src/core/styled_widget.dart
@@ -58,9 +58,6 @@ abstract class StyledWidget extends StatelessWidget {
       builder: builder,
     );
   }
-
-  @override
-  Widget build(BuildContext context);
 }
 
 /// A styled widget that builds its child using a [MixData] object.

diff --git a/packages/mix/lib/src/specs/icon/icon_widget.dart b/packages/mix/lib/src/specs/icon/icon_widget.dart
@@ -15,8 +15,7 @@ class StyledIcon extends StyledWidget {
     @Deprecated('Use orderOfModifiers parameter instead')
     List<Type> modifierOrder = const <Type>[],
     List<Type> orderOfModifiers = const <Type>[],
-  })  : assert(modifierOrder == const <Type>[] ||
-            orderOfModifiers == const <Type>[]),
+  })  : assert(modifierOrder.length == 0 || orderOfModifiers.length == 0),
         super(
           orderOfModifiers:
               orderOfModifiers == const [] ? modifierOrder : orderOfModifiers,
@@ -61,10 +60,9 @@ class IconSpecWidget extends StatelessWidget {
     @Deprecated('Use orderOfModifiers parameter instead')
     List<Type> modifierOrder = const <Type>[],
     List<Type> orderOfModifiers = const <Type>[],
-  })  : assert(modifierOrder == const <Type>[] ||
-            orderOfModifiers == const <Type>[]),
+  })  : assert(modifierOrder.length == 0 || orderOfModifiers.length == 0),
         orderOfModifiers =
-            orderOfModifiers == const [] ? modifierOrder : orderOfModifiers;
+            orderOfModifiers.length == 0 ? modifierOrder : orderOfModifiers;
 
   final IconData? icon;
   final IconSpec? spec;
@@ -105,12 +103,10 @@ class AnimatedStyledIcon extends StyledWidget {
     @Deprecated('Use orderOfModifiers parameter instead')
     List<Type> modifierOrder = const <Type>[],
     List<Type> orderOfModifiers = const <Type>[],
-  })  : assert(modifierOrder == const <Type>[] ||
-            orderOfModifiers == const <Type>[]),
+  })  : assert(modifierOrder.length == 0 || orderOfModifiers.length == 0),
         super(
-          orderOfModifiers: orderOfModifiers == const <Type>[]
-              ? modifierOrder
-              : orderOfModifiers,
+          orderOfModifiers:
+              orderOfModifiers.length == 0 ? modifierOrder : orderOfModifiers,
         );
 
   final AnimatedIconData icon;
@@ -148,11 +144,9 @@ class AnimatedIconSpecWidget extends ImplicitlyAnimatedWidget {
     @Deprecated('Use orderOfModifiers parameter instead')
     List<Type> modifierOrder = const <Type>[],
     List<Type> orderOfModifiers = const <Type>[],
-  })  : assert(modifierOrder == const <Type>[] ||
-            orderOfModifiers == const <Type>[]),
-        orderOfModifiers = orderOfModifiers == const <Type>[]
-            ? modifierOrder
-            : orderOfModifiers;
+  })  : assert(modifierOrder.length == 0 || orderOfModifiers.length == 0),
+        orderOfModifiers =
+            orderOfModifiers.length == 0 ? modifierOrder : orderOfModifiers;
 
   final IconData? icon;
   final IconSpec spec;

diff --git a/packages/mix/lib/src/theme/mix/mix_theme.dart b/packages/mix/lib/src/theme/mix/mix_theme.dart
@@ -1,3 +1,4 @@
+import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 
 import '../../internal/iterable_ext.dart';
@@ -152,7 +153,7 @@ class MixThemeData {
         other.breakpoints == breakpoints &&
         other.radii == radii &&
         other.spaces == spaces &&
-        other.defaultOrderOfModifiers == defaultOrderOfModifiers;
+        listEquals(other.defaultOrderOfModifiers, defaultOrderOfModifiers);
   }
 
   @override

diff --git a/packages/mix/lib/src/variants/context_variant.dart b/packages/mix/lib/src/variants/context_variant.dart
@@ -39,9 +39,6 @@ abstract class ContextVariant extends IVariant {
     return VariantAttribute(this, Style.create(params));
   }
 
-  @override
-  bool when(BuildContext context);
-
   @override
   VariantPriority get priority => VariantPriority.normal;