Hi, On Mon, 2 Jun 2014 20:11:51 +0000 Andres Gomez Ramirez <andres.go...@cern.ch> wrote: > I attached a patch for https://labs.riseup.net/code/issues/7344, > "Bug #7344 Double-clicking on an entry in the language or keyboard > layout lists is not enough". > Thanks for the patch.
However, I fail to understand the need of these changes: - if isinstance(data, Gtk.Dialog): - data.response(True) + self.dialog.response(True) - why do you drop the check for a dialog? - why do you use self instead of data (use of data is advised in callbacks as far as I know)? I think that self.dialog should be replaced by data.dialog. The same questions applies for the next change. Cheers _______________________________________________ Tails-dev mailing list Tails-dev@boum.org https://mailman.boum.org/listinfo/tails-dev To unsubscribe from this list, send an empty email to tails-dev-unsubscr...@boum.org.