Review: Approve
Thanks for putting that together quickly.

Remarks
- line 57: the comment is wrong (mentions a font size)
- it looks like the error rollback code could be factored into a helper 
function; but that's fine for a quick fix
- unit tests, good


-- 
https://code.edge.launchpad.net/~macslow/notify-osd/gravity/+merge/10963
Your team ayatana-commits is subscribed to branch lp:notify-osd.

_______________________________________________
Mailing list: https://launchpad.net/~ayatana-commits
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp

Reply via email to