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.
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.
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.googleuser content. com/e_O6xn6quRh 81wFBPFPZK3Ani- M5w0FFY4IPH_ QQ0aaezyPdPPNkJ Cg0OC-3kc_ _bdPU4cgAxw_ WK1aABt2jqpl- Z-t-7iffKPJYIUP kVCS1LjNcm70pMj CY6g. 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.