Closed workspaceking closed 2 years ago
most of the time the header is what we can call a summary of the remaining part of the ui so we need to provide the whole data to the header
I am also willing to contribute
I am developing an app named roadid ( roadid.com ), In this app there is a history section which has a header ( summary of the history records , like total activities, duration, user since, total distance etc ) which is an aggregate of the history data, that's why sometimes we need it ( to view a summarized data of a data set _
On Sat, Aug 7, 2021 at 6:37 PM Venkatesh Prasad @.***> wrote:
@.**** commented on this pull request.
Hey @rashidcoder https://github.com/rashidcoder thanks for making this PR. I just need to understand few things before merging.
In lib/paginate_firestore.dart https://github.com/vedartm/paginate_firestore/pull/93#discussion_r684633886 :
- SliverPadding(
- padding: widget.padding,
- sliver: SliverList(
- delegate: SliverChildBuilderDelegate(
- (context, index) {
- return;
- },
- ),
- ),
- ),
Can you example why we need this part?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vedartm/paginate_firestore/pull/93#pullrequestreview-724820598, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIOZNGBQYWSR3VAB7J7FFLDT3UZITANCNFSM5BWC4S2A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .
sometime we need to build header from the same data which the builder uses, that's why i added headerBuilder