search

ACM-VIT / fludget

Learn Flutter on Flutter! A widget directory with implementation samples!
MIT License
31 stars 63 forks source link

Added Padding widget #276

Closed yuran09 closed 3 years ago

yuran09 commented 3 years ago

I've added the padding widget with some examples of application

https://user-images.githubusercontent.com/33298980/138355795-0f3feec1-d597-43d9-b66c-eb88ec0394bf.mp4

acmbot[bot] commented 3 years ago

Thanks @yuran09, for opening the pull request! 🙌 One of our team-mates will review the pull request soon. ✅

yuran09 commented 3 years ago

Will do

yuran09 commented 3 years ago

As mentioned in the issue #268, please include one example without using the widget at the start as well to showcase the benefit of the widget

All requested changes made, awaiting merge

AryanKhubchandani commented 3 years ago

Can you please share a screen recording of the changes made for review?

yuran09 commented 3 years ago

Can you please share a screen recording of the changes made for review?

Ok

yuran09 commented 3 years ago

Can you please share a screen recording of the changes made for review?

https://user-images.githubusercontent.com/33298980/138517651-ec068548-0f12-4ced-bcc7-9d934b314feb.mp4

Please check it before someone make a merge and then i'll have conflits

acmbot[bot] commented 3 years ago

Congratulations @yuran09, your pull request is merged! 🎉 Thanks for your contributions. 🙌

AryanKhubchandani commented 3 years ago

Please check it before someone make a merge and then i'll have conflits

Don't worry about that, if the conflicts have been created because of me merging some other PR before yours, I solve the conflicts myself only.