Adding names to the Widget's builder parameters

[MoathCodes]

Hello,

I didn't think opening an issue was the right approach, so I created a discussion instead.

This is a small quality-of-life improvement for widgets that accept a builder (or similar) parameter: give the builder parameters descriptive names so the language server's autocomplete shows helpful names instead of p0, p1, etc.

For example, currently you might see:

showFDialog(
  context: context,
  builder: (p0, p1, p2) => const _StartWizardDialog(),
);

But by changing the declaration from this:

Future<T?> showFDialog<T>({
  required BuildContext context,
  required Widget Function(BuildContext, FDialogStyle, Animation<double>) builder,
  ...
})

to this:

Future<T?> showFDialog<T>({
  required BuildContext context,
  required Widget Function(
    BuildContext context,
    FDialogStyle style,
    Animation<double> animation,
  ) builder,
  ...
})

autocomplete will produce a much clearer builder:

showFDialog(
  context: context,
  builder: (context, style, animation) => const _StartWizardDialog(),
);

It’s a tiny change, but as someone trying ForUI for the first time it makes the API much easier to understand. If these changes to widgets with the same issue would be welcome, I’d be happy to submit a PR.

[Pante]

Sure thing! A PR would be awesome.

[Pante]

@MoathCodes Just a note but I'm currently reworking forui_internal_gen & adding names to the parameters for generated functions like copyWith.