Code review comment for lp:~om26er/ubuntu-calendar-app/autopilot_tests_bootstrap

Revision history for this message
Olivier Tilloy (osomon) wrote :

224 + x, y, h, w = toolbar.globalRect
225 + tx = x + (h / 2)
226 + ty = y + (w - 2)

If I’m not mistaken the globalRect tuple contains, in this order: x, y, width, height. Using 'h' for width and 'w' for height is very confusing.

review: Needs Fixing

« Back to merge proposal