Provide onTap function for BasicEvent type or add a BasicTouchEvent type about timetable

[JonasWanke]

Thanks for the feature request! That's definitely a nice feature to have, I'll probably implement it tomorrow morning

[JonasWanke]

This is now implemented in v0.2.2. For a cleaner separation of concerns (data vs. UI), the onTap-parameter can be specified in BasicEventWidget & BasicAllDayEventWidget as opposed to specifying it in BasicEvent.

[JonasWanke]

While all taps invoke the same lambda, you know the event at creation time of the BasicEventWidget and using code like the following you can also access it in the onTap-handler:

Timetable<BasicEvent>(
  eventBuilder: (event) {
    return BasicEventWidget(
      event,
      onTap: () {
        if (event.something) {
          // ...
        }
      },
      // or like this:
      onTap: () => myOnTapHandler(event),
    );
  },
);

void myOnTapHandler(BasicEvent event) {
  // ...
}

I hope this answers your question.

[raLaaaa]

After thinking a while about this my mind raised a question. Adding the onTap parameter to the BasicEventWidget sounds legit regarding the seperation of concerns. How ever doesnt this lead to the fact that regardless of the event the onTap` will trigger the same method?

So what would be the solution if I need different callbacks depending on the event? I hope my question is clear.