Mercurial > pidgin
annotate src/gtkeventloop.c @ 11016:6417b2f5de4e
[gaim-migrate @ 12885]
Wherever possible, allow users to act on objects and data directly, rather
than through dialogs or explicit commands. For example, it is more intuitive
to drag a circle object around in a diagram rather than selecting a "Move"
command from a menu while the circle is selected. Simlarly, in an email
application, allow the user to attach files by dragging them from the file
manager and dropping them onto the message composition window if they wish.
-- GNOME HIG
This allows direct manipulation of buddy aliases and group names, by moving
the features from dialogs to GtkTreeView's inline editing.
I think this is a great change, but it seems like the type of thing that
might stir up controversy. If so, take it to the mailing list for discussion.
If not, I'll go ahead and perfect this sometime later.
-s.
committer: Tailor Script <tailor@pidgin.im>
| author | Sean Egan <seanegan@gmail.com> |
|---|---|
| date | Tue, 21 Jun 2005 04:40:34 +0000 |
| parents | 482fc53c969d |
| children | 8e97c8befc0b |
| rev | line source |
|---|---|
| 8273 | 1 /** |
| 2 * @file gtk_eventloop.c Gaim Event Loop API (gtk implementation) | |
| 3 * @ingroup gtkui | |
| 4 * | |
| 5 * gaim | |
| 6 * | |
| 7 * Gaim is the legal property of its developers, whose names are too numerous | |
| 8 * to list here. Please refer to the COPYRIGHT file distributed with this | |
| 9 * source distribution. | |
| 10 * | |
| 11 * This program is free software; you can redistribute it and/or modify | |
| 12 * it under the terms of the GNU General Public License as published by | |
| 13 * the Free Software Foundation; either version 2 of the License, or | |
| 14 * (at your option) any later version. | |
| 15 * | |
| 16 * This program is distributed in the hope that it will be useful, | |
| 17 * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 18 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 19 * GNU General Public License for more details. | |
| 20 * | |
| 21 * You should have received a copy of the GNU General Public License | |
| 22 * along with this program; if not, write to the Free Software | |
| 23 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA | |
| 24 */ | |
| 25 | |
| 26 #include <glib.h> | |
| 27 #include "gtkeventloop.h" | |
| 28 #include "eventloop.h" | |
| 29 | |
| 30 #define GAIM_GTK_READ_COND (G_IO_IN | G_IO_HUP | G_IO_ERR) | |
| 31 #define GAIM_GTK_WRITE_COND (G_IO_OUT | G_IO_HUP | G_IO_ERR | G_IO_NVAL) | |
| 32 | |
| 33 typedef struct _GaimGtkIOClosure { | |
| 34 GaimInputFunction function; | |
| 35 guint result; | |
| 36 gpointer data; | |
| 37 | |
| 38 } GaimGtkIOClosure; | |
| 39 | |
| 40 static void gaim_gtk_io_destroy(gpointer data) | |
| 41 { | |
| 42 g_free(data); | |
| 43 } | |
| 44 | |
| 45 static gboolean gaim_gtk_io_invoke(GIOChannel *source, GIOCondition condition, gpointer data) | |
| 46 { | |
| 47 GaimGtkIOClosure *closure = data; | |
| 48 GaimInputCondition gaim_cond = 0; | |
| 49 | |
| 50 if (condition & GAIM_GTK_READ_COND) | |
| 51 gaim_cond |= GAIM_INPUT_READ; | |
| 52 if (condition & GAIM_GTK_WRITE_COND) | |
| 53 gaim_cond |= GAIM_INPUT_WRITE; | |
| 54 | |
| 55 #if 0 | |
| 56 gaim_debug(GAIM_DEBUG_MISC, "gtk_eventloop", | |
| 57 "CLOSURE: callback for %d, fd is %d\n", | |
| 58 closure->result, g_io_channel_unix_get_fd(source)); | |
| 59 #endif | |
| 60 | |
| 61 closure->function(closure->data, g_io_channel_unix_get_fd(source), gaim_cond); | |
| 62 | |
| 63 return TRUE; | |
| 64 } | |
| 65 | |
|
8280
084ed9f7ac19
[gaim-migrate @ 9004]
Christian Hammond <chipx86@chipx86.com>
parents:
8273
diff
changeset
|
66 static guint gaim_gtk_input_add(gint fd, GaimInputCondition condition, GaimInputFunction function, |
| 8273 | 67 gpointer data) |
| 68 { | |
| 69 GaimGtkIOClosure *closure = g_new0(GaimGtkIOClosure, 1); | |
| 70 GIOChannel *channel; | |
| 71 GIOCondition cond = 0; | |
| 72 | |
| 73 closure->function = function; | |
| 74 closure->data = data; | |
| 75 | |
| 76 if (condition & GAIM_INPUT_READ) | |
| 77 cond |= GAIM_GTK_READ_COND; | |
| 78 if (condition & GAIM_INPUT_WRITE) | |
| 79 cond |= GAIM_GTK_WRITE_COND; | |
| 80 | |
|
8280
084ed9f7ac19
[gaim-migrate @ 9004]
Christian Hammond <chipx86@chipx86.com>
parents:
8273
diff
changeset
|
81 channel = g_io_channel_unix_new(fd); |
| 8273 | 82 closure->result = g_io_add_watch_full(channel, G_PRIORITY_DEFAULT, cond, |
| 83 gaim_gtk_io_invoke, closure, gaim_gtk_io_destroy); | |
| 84 | |
| 85 #if 0 | |
| 86 gaim_debug(GAIM_DEBUG_MISC, "gtk_eventloop", | |
| 87 "CLOSURE: adding input watcher %d for fd %d\n", | |
|
8280
084ed9f7ac19
[gaim-migrate @ 9004]
Christian Hammond <chipx86@chipx86.com>
parents:
8273
diff
changeset
|
88 closure->result, fd); |
| 8273 | 89 #endif |
| 90 | |
| 91 g_io_channel_unref(channel); | |
| 92 return closure->result; | |
| 93 } | |
| 94 | |
| 95 static GaimEventLoopUiOps eventloop_ops = | |
| 96 { | |
| 97 g_timeout_add, | |
|
8470
9949b752d1ab
[gaim-migrate @ 9203]
Christian Hammond <chipx86@chipx86.com>
parents:
8287
diff
changeset
|
98 (guint (*)(guint))g_source_remove, |
| 8273 | 99 gaim_gtk_input_add, |
| 8807 | 100 (guint (*)(guint))g_source_remove |
| 8273 | 101 }; |
| 102 | |
| 103 GaimEventLoopUiOps * | |
| 104 gaim_gtk_eventloop_get_ui_ops(void) | |
| 105 { | |
| 106 return &eventloop_ops; | |
| 107 } |
