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

Revision history for this message
Lars Karlitski (larsu) wrote :

For the sake of consistency, I'd recommend not doing this kind of change in applications, but in gtk+ itself.

Approving anyway because this fixes a specific bug and changing gtk+ requires more thought. Also, the comment clearly states that this is a hack ;)

I've filed bug #953757 so that we don't lose track of the issue.

Good Work!

review: Approve

« Back to merge proposal