Fix board editor color filter

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 6.2.3
+
+- Fix board editor color filter
+
 ## 6.2.2
 
 - Fix default value of brightness

example/pubspec.lock

@@ -31,7 +31,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "6.2.2"
+    version: "6.2.3"
   clock:
     dependency: transitive
     description:

lib/src/widgets/board_editor.dart

@@ -232,26 +232,20 @@ class _BoardEditorState extends State<ChessboardEditor> {
       ),
     );
 
-    final coloredBoard = BrightnessHueFilter(
-      hue: widget.settings.hue,
-      brightness: widget.settings.brightness,
-      child: board,
-    );
-
-    if (widget.settings.border != null) {
-      return BorderedChessboard(
-        size: widget.size,
-        orientation: widget.orientation,
-        border: widget.settings.border!,
-        showCoordinates: widget.settings.enableCoordinates,
-        child: coloredBoard,
-      );
-    }
+    final borderedChessboard = widget.settings.border != null
+        ? BorderedChessboard(
+            size: widget.size,
+            orientation: widget.orientation,
+            border: widget.settings.border!,
+            showCoordinates: widget.settings.enableCoordinates,
+            child: board,
+          )
+        : board;
 
     return BrightnessHueFilter(
       brightness: widget.settings.brightness,
       hue: widget.settings.hue,
-      child: coloredBoard,
+      child: borderedChessboard,
     );
   }
 

pubspec.yaml

@@ -1,6 +1,6 @@
 name: chessground
 description: Chess board UI developed for lichess.org. It has no chess logic inside so it can be used for chess variants.
-version: 6.2.2
+version: 6.2.3
 repository: https://github.com/lichess-org/flutter-chessground
 funding:
   - https://lichess.org/patron

test/widgets/board_editor_test.dart

@@ -56,6 +56,60 @@ void main() {
       expect(find.byType(PieceWidget), findsNWidgets(12));
     });
 
+    testWidgets('displays a border', (WidgetTester tester) async {
+      const board = ChessboardEditor(
+        size: boardSize,
+        orientation: Side.white,
+        pieces: {},
+        settings: ChessboardSettings(
+          border: BoardBorder(
+            width: 16.0,
+            color: Color(0xFF000000),
+          ),
+        ),
+      );
+
+      await tester.pumpWidget(board);
+
+      final size = tester.getSize(
+        find.byType(SolidColorChessboardBackground),
+      );
+      expect(size.width, boardSize - 32.0);
+      expect(size.height, boardSize - 32.0);
+    });
+
+    testWidgets('change in hue will use a color filter',
+        (WidgetTester tester) async {
+      const board = ChessboardEditor(
+        size: boardSize,
+        orientation: Side.white,
+        pieces: {},
+        settings: ChessboardSettings(
+          hue: 100.0,
+        ),
+      );
+
+      await tester.pumpWidget(board);
+
+      expect(find.byType(ColorFiltered), findsOneWidget);
+    });
+
+    testWidgets('change in brightness will use a color filter',
+        (WidgetTester tester) async {
+      const board = ChessboardEditor(
+        size: boardSize,
+        orientation: Side.white,
+        pieces: {},
+        settings: ChessboardSettings(
+          brightness: 0.9,
+        ),
+      );
+
+      await tester.pumpWidget(board);
+
+      expect(find.byType(ColorFiltered), findsOneWidget);
+    });
+
     testWidgets(
         'touching a square triggers the onEditedSquare callback when the board pointer tool mode is `edit`',
         (WidgetTester tester) async {

test/widgets/board_test.dart

@@ -70,6 +70,38 @@ void main() {
       expect(size.width, boardSize - 32.0);
       expect(size.height, boardSize - 32.0);
     });
+
+    testWidgets('change in hue will use a color filter',
+        (WidgetTester tester) async {
+      const board = Chessboard.fixed(
+        size: boardSize,
+        orientation: Side.white,
+        fen: kInitialFEN,
+        settings: ChessboardSettings(
+          hue: 100.0,
+        ),
+      );
+
+      await tester.pumpWidget(board);
+
+      expect(find.byType(ColorFiltered), findsOneWidget);
+    });
+
+    testWidgets('change in brightness will use a color filter',
+        (WidgetTester tester) async {
+      const board = Chessboard.fixed(
+        size: boardSize,
+        orientation: Side.white,
+        fen: kInitialFEN,
+        settings: ChessboardSettings(
+          brightness: 0.9,
+        ),
+      );
+
+      await tester.pumpWidget(board);
+
+      expect(find.byType(ColorFiltered), findsOneWidget);
+    });
   });
 
   group('Interactive board', () {