Here is a report of my investigation and my chat with Marco this afternoon:
The problem is topChanged and bottomChanged are connected to _q_relayout(). The relayout function does the relayout of the element, it doesn't update the height of the layout. They should instead trigger _q_updateSize(), that will call _q_relayout() after updating the size. There were already functionality and unit tests in place that ensure that the height is updated whenever the slot-specific paddings change. There was a test checking that having the "global" top/bottom padding *initialized* to a *custom* value would produce a layout with the correct height, but there was no test checking that *changing * the global padding *after* initialization would update the layout height. Well spotted, and sorry for the inconvenience, Marco! Thanks a ton for the fix and the unit test! :) -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to ubuntu-ui-toolkit in Ubuntu. https://bugs.launchpad.net/bugs/1628161 Title: SlotsLayout height is not dynamically updated when main paddings change Status in ubuntu-ui-toolkit package in Ubuntu: In Progress Bug description: You can see this in the attached testcase. When paddings are changed, the SlotsLayout height is not updated dynamically. To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1628161/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
