Code review comment for lp:~carla-sella/ubuntu-docviewer-app/test-toc

Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

My Autopilot knowledge is poor, I'm learning case by case.
Dunno how to make test waiting for the bottomEdgeTip to be completely visible, but I found a better workaround that worked for me.

Since the tip is always partially visible (by a units.gu(1) offset), even when hidden, setting
    start_y = (action_item.globalRect.y + 1)
does the trick and grants a legal value which is inside the window boundaries.

The issue with scroll seems to be the same issue I have with the bottomEdge.
I tried to add a further check that ensure the ToC entry will be visible, and it worked: http://paste.ubuntu.com/10985020/

« Back to merge proposal