yellowQ-software / yellowQ-Flutter-Image-Painter

Flutter package for painting on images
MIT License
108 stars 119 forks source link

Hide each icon in toolbar separately #80

Open AlirezaDaryani opened 6 months ago

AlirezaDaryani commented 6 months ago
lively-bigyan commented 6 months ago

@AlirezaDaryani Thank you for opening the pull request. I do however have doubts about merging these changes. In this last version of 0.7.0, we introduced a new mechanism that exposes the ImagePainterController object. This object allows users to have a high degree of customization if needed because they can create any button and add callback inside the button to do whatever action we previously did on each preset button callback. We are trying to move away from the approach of overloading a single class with parameters for each small action. I do like the fact that the control parameters are consolidated in a single class. But this approach is what we are trying to move away from so it goes against the direction we are headed for the package. Please let me know your thoughts on this.