Code review comment for lp:~charlesk/ido/lp-906050

Revision history for this message
Sebastien Bacher (seb128) wrote :

It works fine for me but I don't know enough of the reason why the offscreen logic was added to start so I'm a bit nervous acking it being dropped, I figure it wouldn't have been added if it was not needed for a reason, I would prefer to have somebody who has a clue about the historical reason to ack it

« Back to merge proposal