search

muhammad369 / ResponsiveGrid_Flutter

Responsive Grid Layout for Flutter
MIT License
88 stars 27 forks source link

[Suggestion] ResponsiveGridList minSpacing as EdgeInsetsGeometry #10

Open Adavo opened 4 years ago

Adavo commented 4 years ago

Hello,

It will be better if minSpacing is not a double but is a EdgeInsetsGeometry. For instance,

minSpacing: 20,

could be translated to

minSpacing: EdgeInsets.all(20),

which will allow us to have different spacing on left-right and top-bottom, for example :

minSpacing: EdgeInsets.fromLTRB(10, 20, 10, 20),

What do you think about this idea ? In fact I encountered an issue on my app since I think the space between 2 objects on left-right is too large but if I decrease minSpacing, then the space between top and bottom element does not fit...

Thanks