Always put required named parameters first (#115)

analysis_options.yaml

@@ -7,5 +7,6 @@ analyzer:
     unsafe_html: ignore
 linter:
   rules:
+    - always_put_required_named_parameters_first
     - avoid_print
     - avoid_redundant_argument_values

lib/widgets/default_styles.dart

@@ -6,9 +6,9 @@ class QuillStyles extends InheritedWidget {
   final DefaultStyles data;
 
   QuillStyles({
-    Key? key,
     required this.data,
     required Widget child,
+    Key? key,
   }) : super(key: key, child: child);
 
   @override

lib/widgets/editor.dart

@@ -176,14 +176,14 @@ class QuillEditor extends StatefulWidget {
       required this.scrollable,
       required this.padding,
       required this.autoFocus,
-      this.showCursor,
       required this.readOnly,
+      required this.expands,
+      this.showCursor,
       this.placeholder,
       this.enableInteractiveSelection = true,
       this.minHeight,
       this.maxHeight,
       this.customStyles,
-      required this.expands,
       this.textCapitalization = TextCapitalization.sentences,
       this.keyboardAppearance = Brightness.light,
       this.scrollPhysics,

lib/widgets/responsive_widget.dart

@@ -5,12 +5,12 @@ class ResponsiveWidget extends StatelessWidget {
   final Widget? mediumScreen;
   final Widget? smallScreen;
 
-  const ResponsiveWidget(
+  const ResponsiveWidget({
-      {Key? key,
+    required this.largeScreen,
-      required this.largeScreen,
+    this.mediumScreen,
-      this.mediumScreen,
+    this.smallScreen,
-      this.smallScreen})
+    Key? key,
-      : super(key: key);
+  }) : super(key: key);
 
   static bool isSmallScreen(BuildContext context) {
     return MediaQuery.of(context).size.width < 800;

lib/widgets/text_block.dart

@@ -253,11 +253,11 @@ class EditableTextBlock extends StatelessWidget {
 class RenderEditableTextBlock extends RenderEditableContainerBox
     implements RenderEditableBox {
   RenderEditableTextBlock({
-    List<RenderEditableBox>? children,
     required Block block,
     required TextDirection textDirection,
     required EdgeInsetsGeometry padding,
     required Decoration decoration,
+    List<RenderEditableBox>? children,
     ImageConfiguration configuration = ImageConfiguration.empty,
     EdgeInsets contentPadding = EdgeInsets.zero,
   })  : _decoration = decoration,
@@ -558,7 +558,6 @@ class _NumberPoint extends StatelessWidget {
   final double padding;
 
   const _NumberPoint({
-    Key? key,
     required this.index,
     required this.indentLevelCounts,
     required this.count,
@@ -567,6 +566,7 @@ class _NumberPoint extends StatelessWidget {
     required this.attrs,
     this.withDot = true,
     this.padding = 0.0,
+    Key? key,
   }) : super(key: key);
 
   @override
@@ -656,9 +656,9 @@ class _BulletPoint extends StatelessWidget {
   final double width;
 
   const _BulletPoint({
-    Key? key,
     required this.style,
     required this.width,
+    Key? key,
   }) : super(key: key);
 
   @override

lib/widgets/text_line.dart

@@ -27,13 +27,13 @@ class TextLine extends StatelessWidget {
   final EmbedBuilder embedBuilder;
   final DefaultStyles styles;
 
-  const TextLine(
+  const TextLine({
-      {Key? key,
+    required this.line,
-      required this.line,
+    required this.embedBuilder,
-      this.textDirection,
+    required this.styles,
-      required this.embedBuilder,
+    this.textDirection,
-      required this.styles})
+    Key? key,
-      : super(key: key);
+  }) : super(key: key);
 
   @override
   Widget build(BuildContext context) {
@@ -839,7 +839,8 @@ class _TextLineElement extends RenderObjectElement {
   }
 
   @override
-  void moveRenderObjectChild(RenderObject child, dynamic oldSlot, dynamic newSlot) {
+  void moveRenderObjectChild(
+      RenderObject child, dynamic oldSlot, dynamic newSlot) {
     throw UnimplementedError();
   }
 

lib/widgets/text_selection.dart

@@ -245,7 +245,6 @@ class EditorTextSelectionOverlay {
 
 class _TextSelectionHandleOverlay extends StatefulWidget {
   const _TextSelectionHandleOverlay({
-    Key? key,
     required this.selection,
     required this.position,
     required this.startHandleLayerLink,
@@ -255,6 +254,7 @@ class _TextSelectionHandleOverlay extends StatefulWidget {
     required this.onSelectionHandleTapped,
     required this.selectionControls,
     this.dragStartBehavior = DragStartBehavior.start,
+    Key? key,
   }) : super(key: key);
 
   final TextSelection selection;
@@ -468,7 +468,7 @@ class _TextSelectionHandleOverlayState
 
 class EditorTextSelectionGestureDetector extends StatefulWidget {
   const EditorTextSelectionGestureDetector({
-    Key? key,
+    required this.child,
     this.onTapDown,
     this.onForcePressStart,
     this.onForcePressEnd,
@@ -482,7 +482,7 @@ class EditorTextSelectionGestureDetector extends StatefulWidget {
     this.onDragSelectionUpdate,
     this.onDragSelectionEnd,
     this.behavior,
-    required this.child,
+    Key? key,
   }) : super(key: key);
 
   final GestureTapDownCallback? onTapDown;

lib/widgets/toolbar.dart

@@ -25,9 +25,9 @@ class InsertEmbedButton extends StatelessWidget {
   final IconData icon;
 
   const InsertEmbedButton({
-    Key? key,
     required this.controller,
     required this.icon,
+    Key? key,
   }) : super(key: key);
 
   @override
@@ -56,9 +56,9 @@ class LinkStyleButton extends StatefulWidget {
   final IconData? icon;
 
   const LinkStyleButton({
-    Key? key,
     required this.controller,
     this.icon,
+    Key? key,
   }) : super(key: key);
 
   @override
@@ -183,11 +183,11 @@ class ToggleStyleButton extends StatefulWidget {
   final ToggleStyleButtonBuilder childBuilder;
 
   ToggleStyleButton({
-    Key? key,
     required this.attribute,
     required this.icon,
     required this.controller,
     this.childBuilder = defaultToggleStyleButtonBuilder,
+    Key? key,
   }) : super(key: key);
 
   @override
@@ -267,11 +267,11 @@ class ToggleCheckListButton extends StatefulWidget {
   final Attribute attribute;
 
   ToggleCheckListButton({
-    Key? key,
     required this.icon,
     required this.controller,
-    this.childBuilder = defaultToggleStyleButtonBuilder,
     required this.attribute,
+    this.childBuilder = defaultToggleStyleButtonBuilder,
+    Key? key,
   }) : super(key: key);
 
   @override
@@ -371,7 +371,7 @@ Widget defaultToggleStyleButtonBuilder(
 class SelectHeaderStyleButton extends StatefulWidget {
   final QuillController controller;
 
-  const SelectHeaderStyleButton({Key? key, required this.controller})
+  const SelectHeaderStyleButton({required this.controller, Key? key})
       : super(key: key);
 
   @override
@@ -494,14 +494,14 @@ class ImageButton extends StatefulWidget {
 
   final ImageSource imageSource;
 
-  ImageButton(
+  ImageButton({
-      {Key? key,
+    required this.icon,
-      required this.icon,
+    required this.controller,
-      required this.controller,
+    required this.imageSource,
-      required this.imageSource,
+    this.onImagePickCallback,
-      this.onImagePickCallback,
+    this.imagePickImpl,
-      this.imagePickImpl})
+    Key? key,
-      : super(key: key);
+  }) : super(key: key);
 
   @override
   _ImageButtonState createState() => _ImageButtonState();
@@ -600,12 +600,12 @@ class ColorButton extends StatefulWidget {
   final bool background;
   final QuillController controller;
 
-  ColorButton(
+  ColorButton({
-      {Key? key,
+    required this.icon,
-      required this.icon,
+    required this.controller,
-      required this.controller,
+    required this.background,
-      required this.background})
+    Key? key,
-      : super(key: key);
+  }) : super(key: key);
 
   @override
   _ColorButtonState createState() => _ColorButtonState();
@@ -738,12 +738,12 @@ class HistoryButton extends StatefulWidget {
   final bool undo;
   final QuillController controller;
 
-  HistoryButton(
+  HistoryButton({
-      {Key? key,
+    required this.icon,
-      required this.icon,
+    required this.controller,
-      required this.controller,
+    required this.undo,
-      required this.undo})
+    Key? key,
-      : super(key: key);
+  }) : super(key: key);
 
   @override
   _HistoryButtonState createState() => _HistoryButtonState();
@@ -810,12 +810,12 @@ class IndentButton extends StatefulWidget {
   final QuillController controller;
   final bool isIncrease;
 
-  IndentButton(
+  IndentButton({
-      {Key? key,
+    required this.icon,
-      required this.icon,
+    required this.controller,
-      required this.controller,
+    required this.isIncrease,
-      required this.isIncrease})
+    Key? key,
-      : super(key: key);
+  }) : super(key: key);
 
   @override
   _IndentButtonState createState() => _IndentButtonState();
@@ -865,7 +865,7 @@ class ClearFormatButton extends StatefulWidget {
 
   final QuillController controller;
 
-  ClearFormatButton({Key? key, required this.icon, required this.controller})
+  ClearFormatButton({required this.icon, required this.controller, Key? key})
       : super(key: key);
 
   @override
@@ -896,30 +896,31 @@ class _ClearFormatButtonState extends State<ClearFormatButton> {
 class QuillToolbar extends StatefulWidget implements PreferredSizeWidget {
   final List<Widget> children;
 
-  const QuillToolbar({Key? key, required this.children}) : super(key: key);
+  const QuillToolbar({required this.children, Key? key}) : super(key: key);
-
+
-  factory QuillToolbar.basic(
+  factory QuillToolbar.basic({
-      {Key? key,
+    required QuillController controller,
-      required QuillController controller,
+    double toolbarIconSize = 18.0,
-      double toolbarIconSize = 18.0,
+    bool showBoldButton = true,
-      bool showBoldButton = true,
+    bool showItalicButton = true,
-      bool showItalicButton = true,
+    bool showUnderLineButton = true,
-      bool showUnderLineButton = true,
+    bool showStrikeThrough = true,
-      bool showStrikeThrough = true,
+    bool showColorButton = true,
-      bool showColorButton = true,
+    bool showBackgroundColorButton = true,
-      bool showBackgroundColorButton = true,
+    bool showClearFormat = true,
-      bool showClearFormat = true,
+    bool showHeaderStyle = true,
-      bool showHeaderStyle = true,
+    bool showListNumbers = true,
-      bool showListNumbers = true,
+    bool showListBullets = true,
-      bool showListBullets = true,
+    bool showListCheck = true,
-      bool showListCheck = true,
+    bool showCodeBlock = true,
-      bool showCodeBlock = true,
+    bool showQuote = true,
-      bool showQuote = true,
+    bool showIndent = true,
-      bool showIndent = true,
+    bool showLink = true,
-      bool showLink = true,
+    bool showHistory = true,
-      bool showHistory = true,
+    bool showHorizontalRule = false,
-      bool showHorizontalRule = false,
+    OnImagePickCallback? onImagePickCallback,
-      OnImagePickCallback? onImagePickCallback}) {
+    Key? key,
+  }) {
     iconSize = toolbarIconSize;
     return QuillToolbar(key: key, children: [
       Visibility(
@@ -1146,13 +1147,13 @@ class QuillIconButton extends StatelessWidget {
   final double highlightElevation;
 
   const QuillIconButton({
-    Key? key,
     required this.onPressed,
     this.icon,
     this.size = 40,
     this.fillColor,
     this.hoverElevation = 1,
     this.highlightElevation = 1,
+    Key? key,
   }) : super(key: key);
 
   @override
@@ -1184,15 +1185,15 @@ class QuillDropdownButton<T> extends StatefulWidget {
   final ValueChanged<T> onSelected;
 
   const QuillDropdownButton({
-    Key? key,
-    this.height = 40,
-    this.fillColor,
-    this.hoverElevation = 1,
-    this.highlightElevation = 1,
     required this.child,
     required this.initialValue,
     required this.items,
     required this.onSelected,
+    this.height = 40,
+    this.fillColor,
+    this.hoverElevation = 1,
+    this.highlightElevation = 1,
+    Key? key,
   }) : super(key: key);
 
   @override