Comment 2 for bug 1227418

On looking through the design documents, it seems this is the desired behavior. The reason we hardcoded the listheight to some random value units.gu(21) is to ensure that at all times, the listview shows only 3 items. Anything more than 3 items will require the user to scroll the listview. If you are wondering why the listview should show only 3 items, that's because when the user scroll the entire page, the clock face should end up like https://lh5.googleusercontent.com/e_O6xn6quRh81wFBPFPZK3Ani-M5w0FFY4IPH_QQ0aaezyPdPPNkJCg0OC-3kc__bdPU4cgAxw_WK1aABt2jqpl-Z-t-7iffKPJYIUPkVCS1LjNcm70pMjCY6g. For that layout it is not possible to let the listview size to be more than 3 since it will then push the clock face above.

However I do agree that static hardcoding of values is not recommended which results in some bugs that I noticed. As a result, I have a branch at https://code.launchpad.net/~nik90/ubuntu-clock-app/dynamic-positioning-part1, which removes all hardcoded values like this.

Thanks for bringing this to my attention.

I have nonetheless added the UX team to this so that they can make the decision to allow dynamic listview height so that the scrollable inside a scrollable page can be removed. I need design approval before I implement this.