channel_colors [nfc]: Rename StreamColorSwatches to ChannelColorSwatches

Author: Khader-1

lib/widgets/channel_colors.dart

@@ -8,12 +8,12 @@ import 'color.dart';
 
 /// A lazily-computed map from a stream's base color to a
 /// corresponding [ChannelColorSwatch].
-abstract class StreamColorSwatches {
-  /// The [StreamColorSwatches] for the light theme.
-  static final StreamColorSwatches light = _StreamColorSwatchesLight();
+abstract class ChannelColorSwatches {
+  /// The [ChannelColorSwatches] for the light theme.
+  static final ChannelColorSwatches light = _ChannelColorSwatchesLight();
 
-  /// The [StreamColorSwatches] for the dark theme.
-  static final StreamColorSwatches dark = _StreamColorSwatchesDark();
+  /// The [ChannelColorSwatches] for the dark theme.
+  static final ChannelColorSwatches dark = _ChannelColorSwatchesDark();
 
   final Map<int, ChannelColorSwatch> _cache = {};
 
@@ -23,7 +23,7 @@ abstract class StreamColorSwatches {
 
   ChannelColorSwatch _computeForBaseColor(int base);
 
-  /// Gives a [StreamColorSwatches], lerped between [a] and [b] at [t].
+  /// Gives a [ChannelColorSwatches], lerped between [a] and [b] at [t].
   ///
   /// If [a] and [b] are [identical], returns [this].
   ///
@@ -33,37 +33,37 @@ abstract class StreamColorSwatches {
   /// This computation is cached on the instance
   /// in order to save work building [t]'s animation frame when there are
   /// multiple UI elements using the same [subscription.color].
-  static StreamColorSwatches lerp(StreamColorSwatches a, StreamColorSwatches b, double t) {
+  static ChannelColorSwatches lerp(ChannelColorSwatches a, ChannelColorSwatches b, double t) {
     // This short-circuit helps when [a] and [b]
-    // are both [StreamColorSwatches.light]
-    // or both [StreamColorSwatches.dark].
+    // are both [ChannelColorSwatches.light]
+    // or both [ChannelColorSwatches.dark].
     // Empirically, [lerp] is called even when the theme hasn't changed,
     // so this is an important optimization.
     if (identical(a, b)) return a;
 
-    return _StreamColorSwatchesLerped(a, b, t);
+    return _ChannelColorSwatchesLerped(a, b, t);
   }
 }
 
-class _StreamColorSwatchesLight extends StreamColorSwatches {
-  _StreamColorSwatchesLight();
+class _ChannelColorSwatchesLight extends ChannelColorSwatches {
+  _ChannelColorSwatchesLight();
 
   @override
   ChannelColorSwatch _computeForBaseColor(int base) => ChannelColorSwatch.light(base);
 }
 
-class _StreamColorSwatchesDark extends StreamColorSwatches {
-  _StreamColorSwatchesDark();
+class _ChannelColorSwatchesDark extends ChannelColorSwatches {
+  _ChannelColorSwatchesDark();
 
   @override
   ChannelColorSwatch _computeForBaseColor(int base) => ChannelColorSwatch.dark(base);
 }
 
-class _StreamColorSwatchesLerped extends StreamColorSwatches {
-  _StreamColorSwatchesLerped(this.a, this.b, this.t);
+class _ChannelColorSwatchesLerped extends ChannelColorSwatches {
+  _ChannelColorSwatchesLerped(this.a, this.b, this.t);
 
-  final StreamColorSwatches a;
-  final StreamColorSwatches b;
+  final ChannelColorSwatches a;
+  final ChannelColorSwatches b;
   final double t;
 
   @override

lib/widgets/theme.dart

@@ -134,7 +134,7 @@ class DesignVariables extends ThemeExtension<DesignVariables> {
       icon: const Color(0xff666699),
       mainBackground: const Color(0xfff0f0f0),
       title: const Color(0xff1a1a1a),
-      streamColorSwatches: StreamColorSwatches.light,
+      streamColorSwatches: ChannelColorSwatches.light,
       star: const HSLColor.fromAHSL(0.5, 47, 1, 0.41).toColor(),
     );
 
@@ -145,7 +145,7 @@ class DesignVariables extends ThemeExtension<DesignVariables> {
       icon: const Color(0xff7070c2),
       mainBackground: const Color(0xff1d1d1d),
       title: const Color(0xffffffff),
-      streamColorSwatches: StreamColorSwatches.dark,
+      streamColorSwatches: ChannelColorSwatches.dark,
       // TODO(#95) unchanged in dark theme?
       star: const HSLColor.fromAHSL(0.5, 47, 1, 0.41).toColor(),
     );
@@ -177,7 +177,7 @@ class DesignVariables extends ThemeExtension<DesignVariables> {
   final Color title;
 
   // Not exactly from the Figma design, but from Vlad anyway.
-  final StreamColorSwatches streamColorSwatches;
+  final ChannelColorSwatches streamColorSwatches;
 
   // Not named variables in Figma; taken from older Figma drafts, or elsewhere.
   final Color star;
@@ -189,7 +189,7 @@ class DesignVariables extends ThemeExtension<DesignVariables> {
     Color? icon,
     Color? mainBackground,
     Color? title,
-    StreamColorSwatches? streamColorSwatches,
+    ChannelColorSwatches? streamColorSwatches,
     Color? star,
   }) {
     return DesignVariables._(
@@ -214,15 +214,15 @@ class DesignVariables extends ThemeExtension<DesignVariables> {
       icon: Color.lerp(icon, other.icon, t)!,
       mainBackground: Color.lerp(mainBackground, other.mainBackground, t)!,
       title: Color.lerp(title, other.title, t)!,
-      streamColorSwatches: StreamColorSwatches.lerp(streamColorSwatches, other.streamColorSwatches, t),
+      streamColorSwatches: ChannelColorSwatches.lerp(streamColorSwatches, other.streamColorSwatches, t),
       star: Color.lerp(star, other.star, t)!,
     );
   }
 }
 
 /// The theme-appropriate [ChannelColorSwatch] based on [subscription.color].
 ///
-/// For how this value is cached, see [StreamColorSwatches.forBaseColor].
+/// For how this value is cached, see [ChannelColorSwatches.forBaseColor].
 ChannelColorSwatch colorSwatchFor(BuildContext context, Subscription subscription) {
   return DesignVariables.of(context)
     .streamColorSwatches.forBaseColor(subscription.color);

test/widgets/channel_colors_test.dart

@@ -6,9 +6,9 @@ import 'package:zulip/widgets/channel_colors.dart';
 import 'channel_colors_checks.dart';
 
 void main() {
-  group('StreamColorSwatches', () {
+  group('ChannelColorSwatches', () {
     test('.light', () {
-      final instance = StreamColorSwatches.light;
+      final instance = ChannelColorSwatches.light;
 
       const base1 = 0xff76ce90;
       final swatch1 = instance.forBaseColor(base1);
@@ -24,7 +24,7 @@ void main() {
 
     // TODO deduplicate with corresponding light-theme test?
     test('.dark', () {
-      final instance = StreamColorSwatches.dark;
+      final instance = ChannelColorSwatches.dark;
 
       const base1 = 0xff76ce90;
       final swatch1 = instance.forBaseColor(base1);
@@ -40,16 +40,16 @@ void main() {
 
     group('lerp', () {
       test('on identical instances', () {
-        final light = StreamColorSwatches.light;
-        check(StreamColorSwatches.lerp(light, light, 0.5)).identicalTo(light);
+        final light = ChannelColorSwatches.light;
+        check(ChannelColorSwatches.lerp(light, light, 0.5)).identicalTo(light);
 
-        final dark = StreamColorSwatches.dark;
-        check(StreamColorSwatches.lerp(dark, dark, 0.5)).identicalTo(dark);
+        final dark = ChannelColorSwatches.dark;
+        check(ChannelColorSwatches.lerp(dark, dark, 0.5)).identicalTo(dark);
       });
 
       test('from light to dark', () {
-        final instance = StreamColorSwatches
-          .lerp(StreamColorSwatches.light, StreamColorSwatches.dark, 0.4);
+        final instance = ChannelColorSwatches
+          .lerp(ChannelColorSwatches.light, ChannelColorSwatches.dark, 0.4);
 
         const base1 = 0xff76ce90;
         final swatch1 = instance.forBaseColor(base1);