Button Icons used are not configurable

[juliaJKruger]

Hi @pmatatias
Thank you for your library, it is great! I do, however, need one small change - I need to configure the icons.
Now I can set my own widget for the icons, but then my widget overrides the one that is working and the colors don't change. I have forked the project and have created a branch to implement this change without affecting the existing functionality.

Please see the branch below, if everything is in order, can I create a pull request?

skynamo/input-quantity

[pmatatias]

Hi @juliaJKruger ,
I really appreciate your contribution. I just saw your update commit, but I believe that in the current version we can already configure Icon. As you can see the first demo example in the documentation uses the take_off airplane icon.

I remember, your version is the same as the code in input-quantity v:1.x.x. In that version I provided the plusIcon and minusIcon properties directly in the InputQty widget.
but for v2 I refactored the code and moved it as part of QtyDecorationProps

you can customize the icon from the QtyDecorationProps :
https://github.com/pmatatias/input-quantity/blob/7a59cb6117eea277fe94a0e982a029366fec88c7/lib/src/decoration_props.dart#L26C1-L26C1

here the example implementation: example.dart

 decoration: QtyDecorationProps(
    isBordered: false,
    minusBtn: Icon(
      Icons.flight_land_rounded,
      color: Colors.purple,
    ),
    plusBtn:
        Icon(Icons.flight_takeoff, color: Colors.indigo),
  ),

[pmatatias]

@juliaJKruger
But of course I'm open to discussion. I agree that it's easier to configure it in your version, but it's limited to just IconData
If people think these features are good, why don't we implement both. We can configure Icons like your version, and also from QtyDecorationProps for icons as specific widgets (advanced configuration)

[juliaJKruger]

@pmatatias
The problem with using the plusBtn and minusBtn for me is that when you reach the minVal or maxVal the icon doesn't get greyed out as it does when using the default icons. I could write in custom logic to do that when using plusBtn and minusBtn, but I figured since your package can already do that, I just need the icon to be changed.
I didn't want to make any changes that would affect the current functionality, so I'm not suggesting we use my change and remove the advanced configuration buttons, but keep both?

[pmatatias]

I see, this issue is actually on my radar.
The problem is because I not use IconButton that has provided enable dan disable color. So when we provide the custom Icon, it will return the Custom Icon.

actually I was planing to change the GestureDetector in to RawMaterialButton instead , so we can add color for enable, and grayed when its disable directly to the Icon.
RawMaterialButton: https://api.flutter.dev/flutter/material/RawMaterialButton-class.html

It will be great if anyone can help me to refactor the [BuildBtn] widget link here , and use the RawMaterialButton instead.