cleanup code

Author: AhmedAbdoElhawary

lib/features/sorting/base/view/sorting_page.dart

@@ -140,21 +140,14 @@ class _BuildItem extends ConsumerWidget {
     final itemWidth = SortingNotifier.calculateItemWidth(context, size);
     final currentItem = ref.watch(instance.select((state) => state.list[index]));
 
-    final color = currentItem.sortedStatus == SortingStatus.sorted
-        ? SortingNotifier.doneSortingColor
-        : (currentItem.sortedStatus == SortingStatus.swapped
-            ? SortingNotifier.swipedColor
-            : currentItem.sortedStatus == SortingStatus.compared
-                ? SortingNotifier.comparedColor
-                : SortingNotifier.itemColor);
     return Padding(
       padding: EdgeInsets.symmetric(horizontal: SortingNotifier.itemsPadding / 2),
       child: AnimatedContainer(
         duration: speedDuration,
         height: SortingNotifier.calculateItemHeight(item.value, size),
         width: itemWidth,
         decoration: BoxDecoration(
-          color: context.getColor(color),
+          color: context.getColor(currentItem.getColor),
           borderRadius: const BorderRadius.vertical(top: Radius.circular(1)),
         ),
       ),

lib/features/sorting/base/view_model/sorting_notifier.dart

@@ -17,7 +17,7 @@ abstract class SortingNotifier extends StateNotifier<SortingNotifierState> {
 
   static double maxListItemHeight = 250.h;
   static double itemsPadding = 1.w;
-  static const ThemeEnum swipedColor = ThemeEnum.redColor;
+  static const ThemeEnum swipingColor = ThemeEnum.redColor;
   static const ThemeEnum comparedColor = ThemeEnum.comparedColor;
   static const ThemeEnum itemColor = ThemeEnum.blueColor;
   static const ThemeEnum doneSortingColor = ThemeEnum.greenColor;
@@ -145,9 +145,9 @@ abstract class SortingNotifier extends StateNotifier<SortingNotifierState> {
 
           break;
 
-        case SortingStatus.swapped:
-          list[step.index1] = list[step.index1].copyWith(sortedStatus: SortingStatus.swapped);
-          list[step.index2] = list[step.index2].copyWith(sortedStatus: SortingStatus.swapped);
+        case SortingStatus.swiping:
+          list[step.index1] = list[step.index1].copyWith(sortedStatus: SortingStatus.swiping);
+          list[step.index2] = list[step.index2].copyWith(sortedStatus: SortingStatus.swiping);
           state = state.copyWith(list: list);
 
           await Future.delayed(speedDuration);
@@ -162,13 +162,14 @@ abstract class SortingNotifier extends StateNotifier<SortingNotifierState> {
           state = state.copyWith(list: list, positions: positions);
           break;
 
+        case SortingStatus.swiped:
         case SortingStatus.unSorted:
           list[step.index1] = list[step.index1].copyWith(sortedStatus: SortingStatus.unSorted);
           list[step.index2] = list[step.index2].copyWith(sortedStatus: SortingStatus.unSorted);
           state = state.copyWith(list: list);
           break;
 
-      // i don't want to make it green while sorting and mark all of them at once as green at the end
+        // i don't want to make it green while sorting and mark all of them at once as green at the end
         case SortingStatus.sorted:
         case SortingStatus.none:
           list[step.index1] = list[step.index1].copyWith(sortedStatus: SortingStatus.none);
@@ -181,5 +182,6 @@ abstract class SortingNotifier extends StateNotifier<SortingNotifierState> {
 
     await greenSortedItemsAsDone();
   }
+
   List<SortingStep> buildSorting(List<int> values);
 }