Fix polygon tool color picker has no eyedropper button, closes #1121

Author: CodeDoctorDE

api/pubspec.lock

@@ -319,10 +319,10 @@ packages:
     dependency: transitive
     description:
       name: meta
-      sha256: df0c643f44ad098eb37988027a8e2b2b5a031fd3977f06bbfd3a76637e8df739
+      sha256: c82594181e3312f3d0695fc95aaaf7758d75b8d4ae2bbecf223b9fd5109a059d
       url: "https://pub.dev"
     source: hosted
-    version: "1.18.2"
+    version: "1.18.3"
   mime:
     dependency: transitive
     description:

app/lib/handlers/polygon.dart

@@ -448,6 +448,7 @@ class PolygonHandler extends Handler<PolygonTool> with ColoredHandler {
   PreferredSizeWidget? getToolbar(DocumentBloc bloc) {
     final element = _element;
     return PolygonToolbarView(
+      bloc: bloc,
       tool: element == null ? data : data.copyWith(property: element.property),
       hasPoints: (element?.points.length ?? 0) > 0,
       onToolChanged: (tool) => _changeTool(bloc, tool),

app/lib/views/toolbar/polygon.dart

@@ -1,3 +1,5 @@
+import 'package:butterfly/bloc/document_bloc.dart';
+import 'package:butterfly/cubits/current_index.dart';
 import 'package:butterfly/src/generated/i18n/app_localizations.dart';
 import 'package:butterfly/views/toolbar/color.dart';
 import 'package:butterfly_api/butterfly_api.dart';
@@ -9,10 +11,12 @@ class PolygonToolbarView extends StatelessWidget
   final PolygonTool tool;
   final ValueChanged<PolygonTool> onToolChanged;
   final bool hasPoints;
+  final DocumentBloc bloc;
   final VoidCallback? onFinishShape, onSubmit, onDelete;
 
   const PolygonToolbarView({
     super.key,
+    required this.bloc,
     required this.tool,
     required this.onToolChanged,
     required this.onFinishShape,
@@ -28,6 +32,14 @@ class PolygonToolbarView extends StatelessWidget
       onChanged: (value) => onToolChanged.call(
         tool.copyWith(property: tool.property.copyWith(color: value)),
       ),
+      onEyeDropper: (context) {
+        bloc.currentIndexCubit.changeTemporaryHandler(
+          context,
+          EyeDropperTool(),
+          bloc: bloc,
+          temporaryState: TemporaryState.removeAfterRelease,
+        );
+      },
       strokeWidth: tool.property.strokeWidth,
       onStrokeWidthChanged: (value) => onToolChanged.call(
         tool.copyWith(property: tool.property.copyWith(strokeWidth: value)),

metadata/en-US/changelogs/185.txt

@@ -1,2 +1,3 @@
 * Fix PDF spread-to-pages imports rendering as gray placeholders ([#1123](https://github.com/LinwoodDev/Butterfly/issues/1123))
-* Fix PDF import using the first pages instead of the selected pages ([#1124](https://github.com/LinwoodDev/Butterfly/issues/1124))
+* Fix PDF import using the first pages instead of the selected pages ([#1124](https://github.com/LinwoodDev/Butterfly/issues/1124))
+* Fix polygon tool color picker has no eyedropper button ([#1121](https://github.com/LinwoodDev/Butterfly/issues/1121))