Code review comment for lp:~vthompson/ubuntu-calendar-app/fixes-1247191

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

This change doesn’t make sense to me: in PathViewBase, indexType(…) returns 1 for the next year, not for the current one.

On top of that, the root.dragging condition means that whenever the user is dragging, all three delegates will be focused at the same time. How about:

    focus: index == root.currentIndex

review: Needs Fixing

« Back to merge proposal