GTG

Code review comment for lp:~kiddo/gtg/pretty-delete-confirmation

Revision history for this message
Bertrand Rousseau (bertrand-rousseau) wrote :

Hmm, I didn't see that change. You're right, it should stay, but I also think it should also be changed since displaying all the deleted tasks in the dialog text also clutters it. We probably should adopt a behavior similar to gedit's when you close it with several unsaved documents: having a small treeview inside the the dialog when several tasks are going to be removed. By using this, we allow scrolling and we maintain dialog sizes to sane values. But that's another discussion that requires to open a bug. As far as this patch is concerned, this removal forces me to disapprove the patch.

review: Disapprove

« Back to merge proposal