feat: allow to use custom callback QuillToolbarColorButton

CHANGELOG.md

@@ -1,3 +1,7 @@
+## [8.5.2]
+- Update `README.md`
+- Feature: Allow you to add custom callback when the `QuillToolbarColorButton` pressed
+
 ## [8.5.1]
 - Update `README.md`
 

lib/src/models/config/toolbar/buttons/color.dart

@@ -1,6 +1,7 @@
 import 'package:flutter/widgets.dart' show Color;
-import './../../shared_configurations.dart' show QuillSharedConfigurations;
 
+import './../../shared_configurations.dart' show QuillSharedConfigurations;
+import '../../../../widgets/controller.dart';
 import 'base.dart';
 
 class QuillToolbarColorButtonExtraOptions
@@ -31,9 +32,9 @@ class QuillToolbarColorButtonOptions extends QuillToolbarBaseButtonOptions<
     super.afterButtonPressed,
     super.childBuilder,
     super.controller,
-    super.globalIconSize,
     super.iconTheme,
     super.tooltip,
+    this.customOnPressedCallback,
   });
 
   final double? iconSize;
@@ -42,4 +43,11 @@ class QuillToolbarColorButtonOptions extends QuillToolbarBaseButtonOptions<
   /// By default will use the default `dialogBarrierColor` from
   /// [QuillSharedConfigurations]
   final Color? dialogBarrierColor;
+
+  final QuillToolbarColorPickerOnPressedCallback? customOnPressedCallback;
 }
+
+typedef QuillToolbarColorPickerOnPressedCallback = Future<void> Function(
+  QuillController controller,
+  bool isBackground,
+);

lib/src/models/config/toolbar/buttons/search.dart

@@ -38,10 +38,10 @@ class QuillToolbarSearchButtonOptions extends QuillToolbarBaseButtonOptions {
 
   /// By default we will show simple search dialog ui
   /// you can pass value to this callback to change this
-  final QuillToolbarSearchButtomOnPressedCallback? customOnPressedCallback;
+  final QuillToolbarSearchButtonOnPressedCallback? customOnPressedCallback;
 }
 
-typedef QuillToolbarSearchButtomOnPressedCallback = Future<void> Function(
+typedef QuillToolbarSearchButtonOnPressedCallback = Future<void> Function(
   QuillController controller,
 );
 

lib/src/widgets/toolbar/buttons/color/color.dart

@@ -167,7 +167,16 @@ class QuillToolbarColorButtonState extends State<QuillToolbarColorButton> {
       // if the caller using Cupertino app he might need to wrap the builder
       // with Material() widget
       return childBuilder(
-        options,
+        QuillToolbarColorButtonOptions(
+          afterButtonPressed: afterButtonPressed,
+          dialogBarrierColor: options.dialogBarrierColor,
+          tooltip: tooltip,
+          iconTheme: iconTheme,
+          iconSize: iconSize,
+          iconData: iconData,
+          iconButtonFactor: iconButtonFactor,
+          customOnPressedCallback: options.customOnPressedCallback,
+        ),
         QuillToolbarColorButtonExtraOptions(
           controller: controller,
           context: context,
@@ -206,7 +215,12 @@ class QuillToolbarColorButtonState extends State<QuillToolbarColorButton> {
     );
   }
 
-  void _showColorPicker() {
+  Future<void> _showColorPicker() async {
+    final customCallback = options.customOnPressedCallback;
+    if (customCallback != null) {
+      await customCallback(controller, widget.isBackground);
+      return;
+    }
     showDialog<String>(
       context: context,
       barrierColor: options.dialogBarrierColor ??

lib/src/widgets/toolbar/buttons/search/search.dart

@@ -147,50 +147,4 @@ class QuillToolbarSearchButton extends StatelessWidget {
       ),
     );
   }
-
-  // Those functions ((findText, moveToPosition)) are not ready yet.
-  // but consider moving them to a better place
-  // List<int> _findText({
-  //   required int index,
-  //   required String text,
-  //   required QuillController controller,
-  //   required List<int> offsets,
-  //   required bool wholeWord,
-  //   required bool caseSensitive,
-  //   bool moveToPosition = true,
-  // }) {
-  //   if (text.isEmpty) {
-  //     return List.empty();
-  //   }
-  //   final newOffsets = controller.document.search(
-  //     text,
-  //     caseSensitive: caseSensitive,
-  //     wholeWord: wholeWord,
-  //   );
-  //   index = 0; // TODO: This might need to be updated...
-  //   if (offsets.isNotEmpty && moveToPosition) {
-  //     _moveToPosition(
-  //       index: index,
-  //       text: text,
-  //       controller: controller,
-  //       offsets: offsets,
-  //     );
-  //   }
-  //   return newOffsets;
-  // }
-
-  // void _moveToPosition({
-  //   required int index,
-  //   required String text,
-  //   required QuillController controller,
-  //   required List<int> offsets,
-  // }) {
-  //   controller.updateSelection(
-  //     TextSelection(
-  //       baseOffset: offsets[index],
-  //       extentOffset: offsets[index] + text.length,
-  //     ),
-  //     ChangeSource.LOCAL,
-  //   );
-  // }
 }

lib/src/widgets/toolbar/buttons/select_header_style.dart

@@ -145,8 +145,6 @@ class QuillToolbarSelectHeaderStyleButtonsState
       }
       final theme = Theme.of(context);
       final isSelected = _selectedAttribute == attribute;
-      // TODO: This needs to be updated to materail 3 as well just like in
-      // quill_icon.dart
       return Padding(
         padding: const EdgeInsets.symmetric(horizontal: !kIsWeb ? 1.0 : 5.0),
         child: ConstrainedBox(