Maybe too much loop? about flutter_calendar_carousel HOT 6 CLOSED

Yes, sure that makes sense. I can update it tomorrow for better performance..
Btw @akuma06 What do you think about passing the event list on onDayPressed function? Would it be better?

from flutter_calendar_carousel.

Since the builder won't provide more than 3month's each it won't grow that exponentially. However, what you've been explaining here sounds reasonable although I've not yet looked in the code. @icemanbsi I think @akuma06 wants to hear from you too.

from flutter_calendar_carousel.

@icemanbsi It seem to be a good idea, it can be useful for Stateless widget.

@hyochan Yeah I forgot about this but if, for some reason, someone provides the events for the whole year to the calendar. Even if it only construct the widgets for one month, it need to loop over the whole list of Event for each day cell (if I'm getting it right). It can be an edge case but I think that there is room for improvement here.

from flutter_calendar_carousel.

@akuma06 Yeah. As said I do agree with that part.

from flutter_calendar_carousel.

#34

from flutter_calendar_carousel.

@icemanbsi I think it would be awesome if you could provide us some example of decorating events in readme. I've released the update to 1.3.2.

from flutter_calendar_carousel.