Re: [Tails-dev] FIx Bug #7344 Double-clicking on an entry in…

Delete this message

Reply to this message
Author: Alan
Date:  
To: tails-dev
Subject: Re: [Tails-dev] FIx Bug #7344 Double-clicking on an entry in the language or keyboard layout lists is not enough
Hi,

On Mon, 2 Jun 2014 20:11:51 +0000
Andres Gomez Ramirez <andres.gomez@???> 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