[Telepathy] [PATCH 6/9] Chooser: no need to check if user is editing to catch GDK_Escape

Niv Sardi xaiki at debian.org
Mon Jun 8 13:31:41 PDT 2009


goes like this:
* User presses the entry
* User decides it was a bad idea -> presses escape

It does break a bit the escape to clear, but maybe we should get the entry out of focus so that esc twice will reduce ?

Signed-off-by: Niv Sardi <xaiki at debian.org>
---
 libempathy-gtk/empathy-presence-chooser.c |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/libempathy-gtk/empathy-presence-chooser.c b/libempathy-gtk/empathy-presence-chooser.c
index 8534d91..865285a 100644
--- a/libempathy-gtk/empathy-presence-chooser.c
+++ b/libempathy-gtk/empathy-presence-chooser.c
@@ -560,9 +560,7 @@ presence_chooser_entry_key_press_event_cb (EmpathyPresenceChooser *self,
                                            GdkEventKey            *event,
 					   GtkWidget              *entry)
 {
-	EmpathyPresenceChooserPriv *priv = GET_PRIV (self);
-
-	if (priv->editing_status && event->keyval == GDK_Escape) {
+	if (event->keyval == GDK_Escape) {
 		/* the user pressed Escape, undo the editing */
 		presence_chooser_reset_status (self);
 		return TRUE;
-- 
1.6.3.1



More information about the telepathy mailing list