First step of improving flutter quill extensions

.gitignore

@@ -80,4 +80,7 @@ pubspec.lock
 # For local development
 pubspec_overrides.yaml
 
-old_example
+old_example
+
+# A directory where you put all of your local things that you don't want to push
+.flutter-quill

example/lib/presentation/quill/quill_editor.dart

@@ -7,9 +7,9 @@ import 'package:desktop_drop/desktop_drop.dart' show DropTarget;
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/extensions.dart' show isAndroid, isIOS, isWeb;
 import 'package:flutter_quill/flutter_quill.dart';
-import 'package:flutter_quill_extensions/flutter_quill_extensions.dart';
-import 'package:flutter_quill_extensions/presentation/embeds/widgets/image.dart'
+import 'package:flutter_quill_extensions/embeds/widgets/image.dart'
     show getImageProviderByImageSource, imageFileExtensions;
+import 'package:flutter_quill_extensions/flutter_quill_extensions.dart';
 import 'package:path/path.dart' as path;
 
 import '../extensions/scaffold_messenger.dart';

flutter_quill_extensions/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this project will be documented in this file.
 
+## 0.8.0
+* **Breaking Change**: Completly change the way how the source code structured to more basic and simple way, organize folders and file names, if you use the library
+from `flutter_quill_extensions.dart` then there is nothing you need to do, but if you are using any other import then you need to re-imports
+* Add support for unoffical css property called `deletable` in the image embed, this won't affect how quill js work
+* Improvemenets to the image embed
+
 ## 0.7.2
 * Fix a bug when opening the link dialog for both video and image buttons
 * Update `README.md`

flutter_quill_extensions/lib/presentation/embeds/toolbar/formula_button.dart → flutter_quill_extensions/lib/embeds/formula/toolbar/formula_button.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/flutter_quill.dart';
 
-import '../../models/config/toolbar/buttons/formula.dart';
+import '../../../models/config/toolbar/buttons/formula.dart';
 
 class QuillToolbarFormulaButton extends StatelessWidget {
   const QuillToolbarFormulaButton({

flutter_quill_extensions/lib/embeds/image/editor/image_embed.dart

@@ -0,0 +1,86 @@
+import 'package:flutter/foundation.dart' show kIsWeb;
+import 'package:flutter/material.dart';
+import 'package:flutter_quill/extensions.dart' as base;
+import 'package:flutter_quill/flutter_quill.dart' hide OptionalSize;
+
+import '../../../models/config/editor/image/image.dart';
+import '../../../models/config/shared_configurations.dart';
+import '../../../utils/element_utils/element_utils.dart';
+import '../../widgets/image.dart';
+import 'image_menu.dart';
+
+class QuillEditorImageEmbedBuilder extends EmbedBuilder {
+  QuillEditorImageEmbedBuilder({
+    required this.configurations,
+  });
+  final QuillEditorImageEmbedConfigurations configurations;
+
+  @override
+  String get key => BlockEmbed.imageType;
+
+  @override
+  bool get expanded => false;
+
+  @override
+  Widget build(
+    BuildContext context,
+    QuillController controller,
+    base.Embed node,
+    bool readOnly,
+    bool inline,
+    TextStyle textStyle,
+  ) {
+    assert(!kIsWeb, 'Please provide image EmbedBuilder for Web');
+
+    final imageSource = standardizeImageUrl(node.value.data);
+    final ((imageSize), margin, alignment) = getElementAttributes(node);
+
+    final width = imageSize.width;
+    final height = imageSize.height;
+
+    final image = getImageWidgetByImageSource(
+      imageSource,
+      imageProviderBuilder: configurations.imageProviderBuilder,
+      imageErrorWidgetBuilder: configurations.imageErrorWidgetBuilder,
+      alignment: alignment,
+      height: height,
+      width: width,
+      assetsPrefix: QuillSharedExtensionsConfigurations.get(context: context)
+          .assetsPrefix,
+    );
+
+    final imageSaverService =
+        QuillSharedExtensionsConfigurations.get(context: context)
+            .imageSaverService;
+    return GestureDetector(
+      onTap: configurations.onImageClicked ??
+          () => showDialog(
+                context: context,
+                builder: (_) => QuillProvider.value(
+                  value: context.requireQuillProvider,
+                  child: FlutterQuillLocalizationsWidget(
+                    child: ImageOptionsMenu(
+                      controller: controller,
+                      configurations: configurations,
+                      imageSource: imageSource,
+                      imageSize: imageSize,
+                      isReadOnly: readOnly,
+                      imageSaverService: imageSaverService,
+                    ),
+                  ),
+                ),
+              ),
+      child: Builder(
+        builder: (context) {
+          if (margin != null) {
+            return Padding(
+              padding: EdgeInsets.all(margin),
+              child: image,
+            );
+          }
+          return image;
+        },
+      ),
+    );
+  }
+}

flutter_quill_extensions/lib/presentation/embeds/embed_types/image.dart → flutter_quill_extensions/lib/embeds/image/editor/image_embed_types.dart

@@ -4,8 +4,8 @@ import 'package:flutter/widgets.dart' show BuildContext;
 import 'package:flutter_quill/flutter_quill.dart';
 import 'package:meta/meta.dart' show immutable;
 
-import '../../../logic/extensions/controller.dart';
-import '../../../logic/services/image_picker/s_image_picker.dart';
+import '../../../extensions/controller.dart';
+import '../../../services/image_picker/s_image_picker.dart';
 
 /// When request picking an image, for example when the image button toolbar
 /// clicked, it should be null in case the user didn't choose any image or

flutter_quill_extensions/lib/presentation/embeds/editor/image/image_menu.dart → flutter_quill_extensions/lib/embeds/image/editor/image_menu.dart

@@ -12,10 +12,11 @@ import 'package:flutter_quill/flutter_quill.dart'
         getEmbedNode;
 import 'package:flutter_quill/translations.dart';
 
-import '../../../../logic/models/config/shared_configurations.dart';
-import '../../../../logic/services/image_saver/s_image_saver.dart';
-import '../../../../logic/utils/string.dart';
 import '../../../models/config/editor/image/image.dart';
+import '../../../models/config/shared_configurations.dart';
+import '../../../services/image_saver/s_image_saver.dart';
+import '../../../utils/element_utils/element_utils.dart';
+import '../../../utils/string.dart';
 import '../../../utils/utils.dart';
 import '../../widgets/image.dart' show ImageTapWrapper, getImageStyleString;
 import '../../widgets/image_resizer.dart' show ImageResizer;

flutter_quill_extensions/lib/presentation/embeds/editor/image/image_web.dart → flutter_quill_extensions/lib/embeds/image/editor/image_web_embed.dart

@@ -4,10 +4,10 @@ import 'package:flutter_quill/flutter_quill.dart';
 import 'package:universal_html/html.dart' as html;
 
 import '../../../models/config/editor/image/image_web.dart';
+import '../../../utils/dart_ui/dart_ui_fake.dart'
+    if (dart.library.html) '../../../utils/dart_ui/dart_ui_real.dart' as ui;
+import '../../../utils/element_utils/element_web_utils.dart';
 import '../../../utils/utils.dart';
-import '../../../utils/web_utils.dart';
-import '../shims/dart_ui_fake.dart'
-    if (dart.library.html) '../shims/dart_ui_real.dart' as ui;
 
 class QuillEditorWebImageEmbedBuilder extends EmbedBuilder {
   const QuillEditorWebImageEmbedBuilder({

flutter_quill_extensions/lib/presentation/embeds/toolbar/image_button/image_button.dart → flutter_quill_extensions/lib/embeds/image/toolbar/image_button.dart

@@ -4,11 +4,11 @@ import 'package:flutter/material.dart';
 import 'package:flutter_quill/flutter_quill.dart';
 import 'package:flutter_quill/translations.dart';
 
-import '../../../../logic/models/config/shared_configurations.dart';
-import '../../../../logic/services/image_picker/image_picker.dart';
+import '../../../models/config/shared_configurations.dart';
 import '../../../models/config/toolbar/buttons/image.dart';
-import '../../embed_types/image.dart';
-import '../utils/image_video_utils.dart';
+import '../../../services/image_picker/image_picker.dart';
+import '../../others/image_video_utils.dart';
+import '../editor/image_embed_types.dart';
 import 'select_image_source.dart';
 
 class QuillToolbarImageButton extends StatelessWidget {

flutter_quill_extensions/lib/presentation/embeds/toolbar/image_button/select_image_source.dart → flutter_quill_extensions/lib/embeds/image/toolbar/select_image_source.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/extensions.dart' show isDesktop;
 
-import '../../embed_types/image.dart';
+import '../editor/image_embed_types.dart';
 
 class SelectImageSourceDialog extends StatelessWidget {
   const SelectImageSourceDialog({super.key});

flutter_quill_extensions/lib/presentation/embeds/toolbar/camera_button/camera_button.dart → flutter_quill_extensions/lib/embeds/others/camera_button/camera_button.dart

@@ -8,10 +8,10 @@ import 'package:flutter_quill/flutter_quill.dart'
         QuillToolbarIconButton;
 import 'package:flutter_quill/translations.dart';
 
-import '../../../../logic/models/config/shared_configurations.dart';
-import '../../../../logic/services/image_picker/image_options.dart';
+import '../../../models/config/shared_configurations.dart';
 import '../../../models/config/toolbar/buttons/camera.dart';
-import '../../embed_types/camera.dart';
+import '../../../services/image_picker/image_options.dart';
+import 'camera_types.dart';
 import 'select_camera_action.dart';
 
 class QuillToolbarCameraButton extends StatelessWidget {

flutter_quill_extensions/lib/presentation/embeds/embed_types/camera.dart → flutter_quill_extensions/lib/embeds/others/camera_button/camera_types.dart

@@ -1,8 +1,8 @@
 import 'package:flutter/widgets.dart' show BuildContext;
 import 'package:meta/meta.dart' show immutable;
 
-import 'image.dart';
-import 'video.dart';
+import '../../image/editor/image_embed_types.dart';
+import '../../video/video.dart';
 
 enum CameraAction {
   video,

flutter_quill_extensions/lib/presentation/embeds/toolbar/camera_button/select_camera_action.dart → flutter_quill_extensions/lib/embeds/others/camera_button/select_camera_action.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/translations.dart';
 
-import '../../embed_types/camera.dart';
+import 'camera_types.dart';
 
 class SelectCameraActionDialog extends StatelessWidget {
   const SelectCameraActionDialog({super.key});

flutter_quill_extensions/lib/presentation/embeds/editor/video/video.dart → flutter_quill_extensions/lib/embeds/video/editor/video_embed.dart

@@ -4,6 +4,7 @@ import 'package:flutter_quill/extensions.dart' as base;
 import 'package:flutter_quill/flutter_quill.dart';
 
 import '../../../models/config/editor/video/video.dart';
+import '../../../utils/element_utils/element_utils.dart';
 import '../../../utils/utils.dart';
 import '../../widgets/video_app.dart';
 import '../../widgets/youtube_video_app.dart';

flutter_quill_extensions/lib/presentation/embeds/editor/video/video_web.dart → flutter_quill_extensions/lib/embeds/video/editor/video_web_embed.dart

@@ -5,10 +5,10 @@ import 'package:youtube_player_flutter/youtube_player_flutter.dart'
     show YoutubePlayer;
 
 import '../../../models/config/editor/video/video_web.dart';
+import '../../../utils/dart_ui/dart_ui_fake.dart'
+    if (dart.library.html) '../../../utils/dart_ui/dart_ui_real.dart' as ui;
+import '../../../utils/element_utils/element_web_utils.dart';
 import '../../../utils/utils.dart';
-import '../../../utils/web_utils.dart';
-import '../shims/dart_ui_fake.dart'
-    if (dart.library.html) '../shims/dart_ui_real.dart' as ui;
 
 class QuillEditorWebVideoEmbedBuilder extends EmbedBuilder {
   const QuillEditorWebVideoEmbedBuilder({

flutter_quill_extensions/lib/presentation/embeds/toolbar/video_button/select_video_source.dart → flutter_quill_extensions/lib/embeds/video/toolbar/select_video_source.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/extensions.dart' show isDesktop;
 
-import '../../embed_types/video.dart';
+import '../video.dart';
 
 class SelectVideoSourceDialog extends StatelessWidget {
   const SelectVideoSourceDialog({super.key});

flutter_quill_extensions/lib/presentation/embeds/toolbar/video_button/video_button.dart → flutter_quill_extensions/lib/embeds/video/toolbar/video_button.dart

@@ -3,11 +3,11 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_quill/flutter_quill.dart';
 
-import '../../../../logic/models/config/shared_configurations.dart';
-import '../../../../logic/services/image_picker/image_options.dart';
+import '../../../models/config/shared_configurations.dart';
 import '../../../models/config/toolbar/buttons/video.dart';
-import '../../embed_types/video.dart';
-import '../utils/image_video_utils.dart';
+import '../../../services/image_picker/image_options.dart';
+import '../../others/image_video_utils.dart';
+import '../video.dart';
 import 'select_video_source.dart';
 
 class QuillToolbarVideoButton extends StatelessWidget {

flutter_quill_extensions/lib/presentation/embeds/embed_types/video.dart → flutter_quill_extensions/lib/embeds/video/video.dart

@@ -2,8 +2,8 @@ import 'package:flutter/widgets.dart' show BuildContext;
 import 'package:flutter_quill/flutter_quill.dart';
 import 'package:meta/meta.dart' show immutable;
 
-import '../../../logic/extensions/controller.dart';
-import '../../../logic/services/image_picker/s_image_picker.dart';
+import '../../extensions/controller.dart';
+import '../../services/image_picker/s_image_picker.dart';
 
 /// When request picking an video, for example when the video button toolbar
 /// clicked, it should be null in case the user didn't choose any video or

flutter_quill_extensions/lib/presentation/embeds/widgets/image.dart → flutter_quill_extensions/lib/embeds/widgets/image.dart

@@ -7,7 +7,7 @@ import 'package:photo_view/photo_view.dart';
 
 import '../../models/config/editor/image/image.dart';
 import '../../utils/utils.dart';
-import '../embed_types/image.dart';
+import '../image/editor/image_embed_types.dart';
 
 const List<String> imageFileExtensions = [
   '.jpeg',

flutter_quill_extensions/lib/presentation/embeds/widgets/video_app.dart → flutter_quill_extensions/lib/embeds/widgets/video_app.dart

@@ -6,7 +6,7 @@ import 'package:flutter_quill/flutter_quill.dart';
 import 'package:url_launcher/url_launcher.dart';
 import 'package:video_player/video_player.dart';
 
-import '../../../flutter_quill_extensions.dart';
+import '../../flutter_quill_extensions.dart';
 
 /// Widget for playing back video
 /// Refer to https://github.com/flutter/plugins/tree/master/packages/video_player/video_player

flutter_quill_extensions/lib/logic/extensions/controller.dart → flutter_quill_extensions/lib/extensions/controller.dart

@@ -1,7 +1,5 @@
 import 'package:flutter_quill/flutter_quill.dart';
 
-// ignore: unused_import
-import '../../presentation/embeds/editor/webview.dart';
 import '../utils/quill_image_utils.dart';
 
 /// Extension functions on [QuillController]
@@ -12,27 +10,6 @@ extension QuillControllerExt on QuillController {
   int get index => selection.baseOffset;
   int get length => selection.extentOffset - index;
 
-  /// Insert webview embed block, it requires [initialUrl] to load
-  /// the initial page
-  // void insertWebViewBlock({
-  //   required String initialUrl,
-  // }) {
-  //   final block = BlockEmbed.custom(
-  //     QuillEditorWebViewBlockEmbed(
-  //       initialUrl,
-  //     ),
-  //   );
-
-  //   this
-  //     ..skipRequestKeyboard = true
-  //     ..replaceText(
-  //       index,
-  //       length,
-  //       block,
-  //       null,
-  //     );
-  // }
-
   /// Insert image embed block, it requires the [imageSource]
   ///
   /// it could be local image on the system file