feat: update Flutter Quill and adapt code for new API (#2051)

Author: minikin

catalyst_voices/apps/voices/lib/app/view/app_content.dart

@@ -8,6 +8,7 @@ import 'package:catalyst_voices_localization/catalyst_voices_localization.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_localized_locales/flutter_localized_locales.dart';
+import 'package:flutter_quill/flutter_quill.dart';
 
 const _restorationScopeId = 'rootVoices';
 
@@ -46,6 +47,7 @@ final class _AppContent extends StatelessWidget {
     return const [
       ...VoicesLocalizations.localizationsDelegates,
       LocaleNamesLocalizationsDelegate(),
+      FlutterQuillLocalizations.delegate,
     ];
   }
 

catalyst_voices/apps/voices/lib/widgets/rich_text/voices_rich_text.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: avoid_dynamic_calls
+
 import 'dart:async';
 
 import 'package:catalyst_voices/common/codecs/markdown_codec.dart';
@@ -12,7 +14,7 @@ import 'package:catalyst_voices_shared/catalyst_voices_shared.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_quill/flutter_quill.dart' as quill;
-import 'package:flutter_quill/flutter_quill_internal.dart' as quill_int;
+import 'package:flutter_quill/internal.dart' as quill_int;
 import 'package:flutter_quill_extensions/flutter_quill_extensions.dart'
     as quill_ext;
 
@@ -240,7 +242,7 @@ class _EditorState extends State<_Editor> {
         controller: widget.controller,
         focusNode: widget.focusNode,
         scrollController: widget.scrollController,
-        configurations: quill.QuillEditorConfigurations(
+        config: quill.QuillEditorConfig(
           padding: const EdgeInsets.all(16),
           placeholder: context.l10n.placeholderRichText,
           characterShortcutEvents: quill.standardCharactersShortcutEvents,
@@ -351,9 +353,9 @@ class _Toolbar extends StatelessWidget {
     return Container(
       color: Theme.of(context).colors.onSurfaceNeutralOpaqueLv1,
       padding: const EdgeInsets.symmetric(horizontal: 18),
-      child: quill.QuillToolbar(
-        configurations: const quill.QuillToolbarConfigurations(),
-        child: Row(
+      child: SingleChildScrollView(
+        scrollDirection: Axis.horizontal,
+        child: Wrap(
           children: [
             _ToolbarAttributeIconButton(
               controller: controller,
@@ -407,12 +409,15 @@ class _ToolbarAttributeIconButton extends StatelessWidget {
       controller: controller,
       attribute: attribute,
       options: quill.QuillToolbarToggleStyleButtonOptions(
-        childBuilder: (options, extraOptions) {
+        // TODO(minikin): We need to use dynamic here because
+        // of the bug in the quill package.
+        // https://github.com/singerdmx/flutter-quill/issues/2511
+        childBuilder: (dynamic options, dynamic extraOptions) {
           return _ToolbarIconButton(
             icon: icon,
-            tooltip: options.tooltip,
-            isToggled: extraOptions.isToggled,
-            onPressed: extraOptions.onPressed,
+            tooltip: options.tooltip as String?,
+            isToggled: extraOptions.isToggled as bool,
+            onPressed: extraOptions.onPressed as VoidCallback?,
           );
         },
       ),
@@ -454,12 +459,15 @@ class _ToolbarImageOptionButton extends StatelessWidget {
     return quill_ext.QuillToolbarImageButton(
       controller: controller,
       options: quill_ext.QuillToolbarImageButtonOptions(
-        childBuilder: (options, extraOptions) {
+        // TODO(minikin): We need to use dynamic here because
+        // of the bug in the quill package.
+        // https://github.com/singerdmx/flutter-quill/issues/2511
+        childBuilder: (dynamic options, dynamic extraOptions) {
           return _ToolbarIconButton(
             icon: VoicesAssets.icons.photograph,
-            tooltip: options.tooltip,
+            tooltip: options.tooltip as String?,
             isToggled: false,
-            onPressed: extraOptions.onPressed,
+            onPressed: extraOptions.onPressed as VoidCallback?,
           );
         },
       ),

catalyst_voices/apps/voices/lib/widgets/rich_text/voices_rich_text_rules.dart

@@ -1,6 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/flutter_quill.dart' as quill;
-import 'package:flutter_quill/flutter_quill_internal.dart' as quill_int;
+import 'package:flutter_quill/internal.dart' as quill_int;
 import 'package:flutter_quill/quill_delta.dart';
 
 /// Heuristic rule to exit current block when user inserts two consecutive

catalyst_voices/apps/voices/pubspec.yaml

@@ -47,8 +47,16 @@ dependencies:
   flutter_dropzone_web: 4.0.2
   flutter_localized_locales: ^2.0.5
   flutter_markdown: ^0.7.4+3
-  flutter_quill: ^10.8.2
-  flutter_quill_extensions: ^10.8.2
+  # TODO(minikin): We use our forks as a temporary workaround for F14.
+  flutter_quill:
+    git:
+      url: https://github.com/input-output-hk/flutter-quill
+      ref: 41f37dcf56b97b1bffe10f012dc667795a30e74e
+  flutter_quill_extensions:
+    git:
+      url: https://github.com/input-output-hk/flutter-quill
+      path: flutter_quill_extensions
+      ref: 41f37dcf56b97b1bffe10f012dc667795a30e74e
   flutter_secure_storage: ^9.2.2
   flutter_web_plugins:
     sdk: flutter
@@ -57,7 +65,11 @@ dependencies:
   google_fonts: ^6.2.1
   intl: ^0.19.0
   markdown: ^7.2.2
-  markdown_quill: ^4.2.0
+  # TODO(minikin): We use our forks as a temporary workaround for F14.
+  markdown_quill:
+    git:
+      url: https://github.com/input-output-hk/markdown_quill
+      ref: 059703222cae65bf5ad96ec75049c0c146d76b11
   mask_text_input_formatter: ^2.9.0
   result_type: ^0.2.0
   scrollable_positioned_list: ^0.3.8

catalyst_voices/apps/voices/test/helpers/pump_app.dart

@@ -2,6 +2,8 @@ import 'package:catalyst_voices_brands/catalyst_voices_brands.dart';
 import 'package:catalyst_voices_localization/catalyst_voices_localization.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_localized_locales/flutter_localized_locales.dart';
+import 'package:flutter_quill/flutter_quill.dart'
+    show FlutterQuillLocalizations;
 import 'package:flutter_test/flutter_test.dart';
 
 extension PumpApp on WidgetTester {
@@ -24,6 +26,7 @@ extension PumpApp on WidgetTester {
         localizationsDelegates: const [
           ...VoicesLocalizations.localizationsDelegates,
           LocaleNamesLocalizationsDelegate(),
+          FlutterQuillLocalizations.delegate,
         ],
         supportedLocales: VoicesLocalizations.supportedLocales,
         locale: locale,

catalyst_voices/melos.yaml

@@ -111,15 +111,27 @@ command:
       flutter_dropzone_web: 4.0.2
       flutter_localized_locales: ^2.0.5
       flutter_markdown: ^0.7.4+3
-      flutter_quill: ^10.8.2
-      flutter_quill_extensions: ^10.8.2
+      # TODO(minikin): We use our forks as a temporary workaround for F14.
+      flutter_quill:
+        git:
+          url: https://github.com/input-output-hk/flutter-quill
+          ref: 41f37dcf56b97b1bffe10f012dc667795a30e74e
+      flutter_quill_extensions:
+        git:
+          url: https://github.com/input-output-hk/flutter-quill
+          path: flutter_quill_extensions
+          ref: 41f37dcf56b97b1bffe10f012dc667795a30e74e
       flutter_rust_bridge: 2.5.1
       flutter_secure_storage: ^9.2.2
       formz: ^0.7.0
       intl: ^0.19.0
       logging: ^1.3.0
       markdown: ^7.2.2
-      markdown_quill: ^4.2.0
+      # TODO(minikin): We use our forks as a temporary workaround for F14.
+      markdown_quill:
+        git:
+          url: https://github.com/input-output-hk/markdown_quill
+          ref: 059703222cae65bf5ad96ec75049c0c146d76b11
       meta: ^1.16.0
       result_type: ^0.2.0
       rxdart: ^0.27.7

catalyst_voices/packages/internal/catalyst_voices_assets/assets/document_templates/proposal/F14-Generic/example.proposal.json

@@ -152,7 +152,7 @@
     },
     "pitch": {
         "team": {
-            "who": "Our team consists of experienced blockchain developers with proven track records..."
+            "who": "Our team consists of experienced blockchain developers with proven track records...\n- **This is a test of the image which can be rendered**. ![](https://images.pexels.com/photos/30432517/pexels-photo-30432517/free-photo-of-people-walking-on-historic-railway-platform.jpeg) \n- **This is a test of the image  which can't be rendered**.  ![](https://picography.co/wp-content/uploads/2024/10/picography-rustic-wood-slices-768x1152.jpg)"
         },
         "budget": {
             "costs": "Budget breakdown:\n- Development (70%): 105,000 ADA\n- Testing (15%): 22,500 ADA\n- Documentation (15%): 22,500 ADA"

catalyst_voices/utilities/uikit_example/pubspec.yaml

@@ -30,7 +30,10 @@ dependencies:
     sdk: flutter
   flutter_bloc: ^8.1.5
   flutter_localized_locales: ^2.0.5
-  flutter_quill: ^10.8.2
+  flutter_quill:
+    git:
+      url: https://github.com/input-output-hk/flutter-quill
+      ref: 41f37dcf56b97b1bffe10f012dc667795a30e74e
 
 dev_dependencies:
   build_runner: ^2.4.12