Code review comment for lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/inputFlickableContentWidth

Revision history for this message
Cris Dywan (kalikiana) wrote :

Validation of the state of this MR:

r1694 slightly improves the problem but effectively hides the handlers and moves them offscreen. It can suffer from a binding race where handlers move offscreen in a non-recoverable way.
r1695 removes the Flickable - in theory it simplifies and improves scrolling, in practise it still suffers from the same binding race.
^^ For reference the diff suffers less visibly from the binding race, but it also doesn't improve much on the bug, and the handlers remain largely offscreen as before.

« Back to merge proposal