Code review comment for lp:~renatofilho/address-book-app/fix-1204564

Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

This patch works fine. If you try to save an empty contact, it will indeed be ignored.
I am just wondering if we could disable the save button in case all the fields are blank instead of accepting the action and do nothing. The user would be forced to cancel, and the result of such action would be more clear.

review: Needs Information

« Back to merge proposal