Code review comment for lp:~gang65/ubuntu-calendar-app/ubuntu-calendar-app-quick-add-event

Revision history for this message
Bartosz Kosiorek (gang65) wrote :

> 100 delegate: TimeLineBase {
> 101 @@ -78,7 +75,7 @@
> ...
> 110 @@ -87,6 +84,9 @@
> 111 delegate: comp
> 113
> 114 + TimeLineBackground {
> 115 + }
> 116 +
>
> I think, rather than handling mouse event inside TimeLineBackground, its
> better to handle that inside TimeLineBase.
The problem is that in TimeLineBase I do not have information about hour. I needs that information to set event for specific hour.

« Back to merge proposal