Code review comment for lp:~pkunal-parmar/ubuntu-calendar-app/new_week_day_view

Revision history for this message
Olivier Tilloy (osomon) wrote :

> > Looks good now. I’m not entirely sure I understand why adding one day when
> in
> > day view fixes the issue, but it seems to fix it indeed.
>
> Actually for DayView, TimeLineHeader's startDay is one day less then current
> date or required date.
> So adding one makes it required date.

Got it, that makes sense!

« Back to merge proposal