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

Revision history for this message
Kunal Parmar (pkunal-parmar) wrote :

> OK, I can see the DetailsView, now. Regarding the multiple venues: the data
> model is possible overstretched. Just take the first venue. I'm looking into
> simplifying the data model. There are still bugs with the calendar.qml. You
> need to move the actions to the Page level and there is still a layout issue
> on the desktop, although it looks fine one the touch.

This bug was affecting the layout (https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1160175), I used work around suggested in bug report.

I hope it should work fine now on both device and desktop.

« Back to merge proposal