jayeshpansheriya/awesome_extensions

An extension to the widget that helps to reduce the boilerplate and adds some helpful methods. and you can make responsive design.

awesome_extensions's Issues

Sized Box from number instead of double

Change target of Size[d]BoxExtension from double to num to include also integers.
Also need to cast this while creating the Widget to this as double.

This is currently not possible: 15.heightBox

Use const on Widgets

In Flutter it is recommended to use const constructors whenever possible.
In the padding extensions, for example, it is not used.

  Padding paddingSymmetric(
          {double vertical = 0.0, double horizontal = 0.0, Key? key}) =>
      Padding(
        key: key,
        padding: EdgeInsets.symmetric(
          vertical: vertical,
          horizontal: horizontal,
        ),
        child: this,
      );

This could be:

  Padding paddingSymmetric(
          {double vertical = 0.0, double horizontal = 0.0, Key? key}) =>
      Padding(
        key: key,
        padding: const EdgeInsets.symmetric(
          vertical: vertical,
          horizontal: horizontal,
        ),
        child: this,
      );

Example Extensions in Docs are broken

https://pub.dev/packages/awesome_extensions#theme-extensions

Text('Hello World',style: context.labelSmall.bold) is not valid.
It should be context.labelSmall?.bold.

I hate to be that guy that opens a PR with a docs fix bug. But here you go.
A PR with a single char added: #23

flutter build web --wasm failing

With release of flutter 3.22, we can now build flutter web with wasm

Unfortunately, this package prevents that right now.

Migration guide by flutter team:
https://docs.flutter.dev/platform-integration/web/wasm#js-interop-wasm

Potential wrong arguments in AsyncSnapshotExt

In AsyncSnapshotExt on the method R when<R>. The error callback is using itself as value for Object error param.

In this screenshot, shows that the AsyncSnapshot indeed has an error value (ServerFailure), but the error returned from R Function(Object error, StackTrace? stackTrace) error callback is a closure (the function itself).

String capitalize() error 'Stack overflow'

Using string_extension method capitalize() will trigger a Stack Overflow because of the recursive call to .capitalize() inside.

I believe that's just a small mistake:

/// Capitalize each word inside string
/// Example: your name => Your Name, your name => Your name
String capitalize() {
  if (isEmpty) return this;
  return split(' ').map((e) => e.capitalize()).join(' ');
}

We should use capitalizeFirst() instead of capitalize().

Good work guys, i like the package !

Use getters instead of functions where it makes sense

For instance, I am used to using on a String: myString.isEmpty; having to replace that with myString.isBlank(); is unintuitive and unnecessary

Use generics properly in extensions

For instance
void pop<T>({result}) => Navigator.pop(this, result);
should be
void pop<T extends Object?>([T? result]) => Navigator.pop<T>(this, result);

Also, why result is named here makes no sense to me at all.
There's so many more like this

whenever i use context.pop() my linter always complains about failing to infer type arguments because of this.

[BUG] Extension DateTime: Yesterday not working if different months or years

e.g.:
DateTime instance is 31.01.2023
DateTime.now() is 01.02.2023

Current implemention will return false, that the given instance is yesterday:

bool get isYesterday {
    final nowDate = DateTime.now();
    return year == nowDate.year &&
        month == nowDate.month &&
        day == nowDate.day - 1;
  }

Should replace it with this:

bool get isYesterday {
    final nowDate = DateTime.now();
    final oneDay = const DateTime(days: 1);
    
    return nowDate.subtract(oneDay).isSameDate(this);
  }

PS: Tested it on today's date....

error about nil Widget

The children of MultiChildRenderObjectElement must each has an associated render object.

This typically means that the `Consumer` or its children
are not a subtype of `RenderObjectWidget`.

 Consumer(
                builder: (context, ref, child) {
                  final (progress, fileSize) = ref.watch(isolateDownloadNotifierProvider);
                  if (progress == 1.0 || showAudioControl.value == true) {
                    return Column(
                      children: [
                        Text(
                          "fileSize $fileSize",
                          style: TextStyle(fontSize: 20),
                        ),
                        AnimatedContainer(
                          duration: const Duration(milliseconds: 300),
                          child: ControlPanel(
                            audioPlayer: audioPlayer,
                            showSeekButtons: showSeekButtons.value,
                          ),
                        ),
                      ],
                    );
                  } 
                  } else {
                    return nil;
                  }
                },
              ),

other issues :

https://stackoverflow.com/questions/75870979/flutter-getx-the-children-of-multichildrenderobjectelement-must-each-has-an

ExpandedExtension already define

flexible_extansion.dat and expanded_extension.dart have same name "ExpandedExtension" which ,create conflicts so you have to change the name