aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFederico Mena Quintero <federico@ximian.com>2001-08-15 00:04:33 +0800
committerFederico Mena Quintero <federico@src.gnome.org>2001-08-15 00:04:33 +0800
commitf8473e11f76fef784b136614fb609424e6453911 (patch)
tree23fa33dfad68eebb5add677036a70f214fd887ab
parent9086989f207a11e7edb50177662539bd75da0370 (diff)
downloadgsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.tar
gsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.tar.gz
gsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.tar.bz2
gsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.tar.lz
gsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.tar.xz
gsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.tar.zst
gsoc2013-evolution-f8473e11f76fef784b136614fb609424e6453911.zip
*Please* add accessor functions instead of just object arguments!
2001-08-14 Federico Mena Quintero <federico@ximian.com> *Please* add accessor functions instead of just object arguments! * e-search-bar.c (e_search_bar_set_option_choice): New function. (e_search_bar_set_suboption_choice): New function. (e_search_bar_set_text): New function. (impl_set_arg): Use the functions above instead of setting things directly. (add_dropdown): If the item is a separator, set it as insensitive. (activate_by_subitems): Handle the translate field in the subitem structure. Also, allow the creation of separators by having NULL text strings in the subitems. (set_option): If the item is a separator, set it as insensitive. (set_option): Do not use the subitem_garbage hack. Do proper memory management instead. (e_search_bar_set_suboption): New function to change the suboption items in a search bar. * e-search-bar.h (ESearchBarSubitem): Added a `translate' field. This API sucks so much it is not funny. (ESearchBar): Removed the subitem_garbage hack. Please do proper memory management. svn path=/trunk/; revision=12014
-rw-r--r--widgets/misc/ChangeLog24
-rw-r--r--widgets/misc/e-search-bar.c198
-rw-r--r--widgets/misc/e-search-bar.h10
3 files changed, 190 insertions, 42 deletions
diff --git a/widgets/misc/ChangeLog b/widgets/misc/ChangeLog
index 51a95cc647..ce4994f43f 100644
--- a/widgets/misc/ChangeLog
+++ b/widgets/misc/ChangeLog
@@ -1,3 +1,27 @@
+2001-08-14 Federico Mena Quintero <federico@ximian.com>
+
+ *Please* add accessor functions instead of just object arguments!
+
+ * e-search-bar.c (e_search_bar_set_option_choice): New function.
+ (e_search_bar_set_suboption_choice): New function.
+ (e_search_bar_set_text): New function.
+ (impl_set_arg): Use the functions above instead of setting things
+ directly.
+ (add_dropdown): If the item is a separator, set it as insensitive.
+ (activate_by_subitems): Handle the translate field in the subitem
+ structure. Also, allow the creation of separators by having NULL
+ text strings in the subitems.
+ (set_option): If the item is a separator, set it as insensitive.
+ (set_option): Do not use the subitem_garbage hack. Do proper
+ memory management instead.
+ (e_search_bar_set_suboption): New function to change the suboption
+ items in a search bar.
+
+ * e-search-bar.h (ESearchBarSubitem): Added a `translate' field.
+ This API sucks so much it is not funny.
+ (ESearchBar): Removed the subitem_garbage hack. Please do proper
+ memory management.
+
2001-08-14 Jon Trowbridge <trow@ximian.com>
* e-search-bar.c (activate_by_subitems): Oops... initial
diff --git a/widgets/misc/e-search-bar.c b/widgets/misc/e-search-bar.c
index ddaf7ed66a..ad5e07d523 100644
--- a/widgets/misc/e-search-bar.c
+++ b/widgets/misc/e-search-bar.c
@@ -162,7 +162,19 @@ activate_by_subitems (ESearchBar *esb, gint item_id, ESearchBarSubitem *subitems
esb->suboption_menu = menu = gtk_menu_new ();
for (i = 0; subitems[i].id != -1; ++i) {
- menu_item = gtk_menu_item_new_with_label (_(subitems[i].text));
+ if (subitems[i].text) {
+ char *str;
+
+ if (subitems[i].translate)
+ str = _(subitems[i].text);
+ else
+ str = subitems[i].text;
+
+ menu_item = gtk_menu_item_new_with_label (str);
+ } else {
+ menu_item = gtk_menu_item_new ();
+ gtk_widget_set_sensitive (menu_item, FALSE);
+ }
gtk_object_set_data (GTK_OBJECT (menu_item), "EsbItemId", GINT_TO_POINTER (item_id));
gtk_object_set_data (GTK_OBJECT (menu_item), "EsbSubitemId", GINT_TO_POINTER (subitems[i].id));
@@ -257,19 +269,6 @@ copy_subitems (ESearchBarSubitem *subitems)
}
static void
-free_subitems (ESearchBarSubitem *subitems)
-{
- gint i;
-
- if (subitems != NULL) {
- for (i=0; subitems[i].id != -1; ++i) {
- g_free (subitems[i].text);
- }
- g_free (subitems);
- }
-}
-
-static void
add_dropdown (ESearchBar *esb, ESearchBarItem *items)
{
GtkWidget *menu = esb->dropdown_menu;
@@ -283,9 +282,10 @@ add_dropdown (ESearchBar *esb, ESearchBarItem *items)
item = e_utf8_gtk_menu_item_new_with_label (GTK_MENU (menu), str);
} else
item = gtk_menu_item_new_with_label (str);
- }
- else
+ } else {
item = gtk_menu_item_new();
+ gtk_widget_set_sensitive (item, FALSE);
+ }
gtk_widget_show (item);
gtk_menu_append (GTK_MENU (menu), item);
@@ -328,6 +328,37 @@ set_dropdown (ESearchBar *esb,
}
}
+/* Frees an array of subitem information */
+static void
+free_subitems (ESearchBarSubitem *subitems)
+{
+ ESearchBarSubitem *s;
+
+ g_assert (subitems != NULL);
+
+ for (s = subitems; s->id != -1; s++) {
+ if (s->text)
+ g_free (s->text);
+ }
+
+ g_free (subitems);
+}
+
+/* Callback used when an option item is destroyed. We have to destroy its
+ * suboption items.
+ */
+static void
+option_item_destroy_cb (GtkObject *object, gpointer data)
+{
+ ESearchBarSubitem *subitems;
+
+ subitems = data;
+
+ g_assert (subitems != NULL);
+ free_subitems (subitems);
+ gtk_object_set_data (object, "EsbChoiceSubitems", NULL);
+}
+
static void
set_option (ESearchBar *esb, ESearchBarItem *items)
{
@@ -357,9 +388,10 @@ set_option (ESearchBar *esb, ESearchBarItem *items)
item = e_utf8_gtk_menu_item_new_with_label (GTK_MENU (menu), str);
} else
item = gtk_menu_item_new_with_label (str);
- }
- else
+ } else {
item = gtk_menu_item_new ();
+ gtk_widget_set_sensitive (item, FALSE);
+ }
gtk_menu_append (GTK_MENU (menu), item);
@@ -367,14 +399,13 @@ set_option (ESearchBar *esb, ESearchBarItem *items)
if (items[i].subitems != NULL) {
subitems = copy_subitems (items[i].subitems);
- esb->subitem_garbage = g_list_prepend (esb->subitem_garbage, subitems);
+ gtk_object_set_data (GTK_OBJECT (item), "EsbChoiceSubitems", subitems);
+ gtk_signal_connect (GTK_OBJECT (item), "destroy",
+ GTK_SIGNAL_FUNC (option_item_destroy_cb), subitems);
}
- gtk_object_set_data (GTK_OBJECT (item), "EsbChoiceSubitems", subitems);
-
- if (i == 0) {
+ if (i == 0)
activate_by_subitems (esb, items[i].id, subitems);
- }
gtk_signal_connect (GTK_OBJECT (item), "activate",
GTK_SIGNAL_FUNC (option_activated_cb),
@@ -476,30 +507,18 @@ static void
impl_set_arg (GtkObject *object, GtkArg *arg, guint arg_id)
{
ESearchBar *esb = E_SEARCH_BAR(object);
- int row;
switch (arg_id) {
case ARG_OPTION_CHOICE:
- esb->option_choice = GTK_VALUE_ENUM(*arg);
- row = find_id (esb->option_menu, esb->option_choice, "EsbChoiceId", NULL);
- if (row == -1)
- row = 0;
- gtk_option_menu_set_history (GTK_OPTION_MENU (esb->option), row);
- emit_query_changed (esb);
+ e_search_bar_set_option_choice (esb, GTK_VALUE_ENUM (*arg));
break;
case ARG_SUBOPTION_CHOICE:
- esb->suboption_choice = GTK_VALUE_ENUM (*arg);
- row = find_id (esb->suboption_menu, esb->suboption_choice, "EsbSubitemId", NULL);
- if (row == -1)
- row = 0;
- gtk_option_menu_set_history (GTK_OPTION_MENU (esb->suboption), row);
- emit_query_changed (esb);
+ e_search_bar_set_suboption_choice (esb, GTK_VALUE_ENUM (*arg));
break;
case ARG_TEXT:
- e_utf8_gtk_editable_set_text (GTK_EDITABLE (esb->entry), GTK_VALUE_STRING (*arg));
- emit_query_changed (esb);
+ e_search_bar_set_text (esb, GTK_VALUE_STRING (*arg));
break;
default:
@@ -522,9 +541,6 @@ impl_destroy (GtkObject *object)
if (esb->suboption)
gtk_object_unref (GTK_OBJECT (esb->suboption));
- g_list_foreach (esb->subitem_garbage, (GFunc) free_subitems, NULL);
- g_list_free (esb->subitem_garbage);
-
if (esb->pending_change) {
gtk_idle_remove (esb->pending_change);
esb->pending_change = 0;
@@ -667,6 +683,50 @@ e_search_bar_set_option (ESearchBar *search_bar, ESearchBarItem *option_items)
((ESearchBarClass *)((GtkObject *)search_bar)->klass)->set_option (search_bar, option_items);
}
+/**
+ * e_search_bar_set_suboption:
+ * @search_bar: A search bar.
+ * @option_id: Identifier of the main option menu item under which the subitems
+ * are to be set.
+ * @subitems: Array of subitem information.
+ *
+ * Sets the items for the secondary option menu of a search bar.
+ **/
+void
+e_search_bar_set_suboption (ESearchBar *search_bar, int option_id, ESearchBarSubitem *subitems)
+{
+ int row;
+ GtkWidget *item;
+ ESearchBarSubitem *old_subitems;
+ ESearchBarSubitem *new_subitems;
+
+ g_return_if_fail (search_bar != NULL);
+ g_return_if_fail (E_IS_SEARCH_BAR (search_bar));
+
+ row = find_id (search_bar->option_menu, option_id, "EsbChoiceId", &item);
+ g_return_if_fail (row != -1);
+ g_assert (item != NULL);
+
+ old_subitems = gtk_object_get_data (GTK_OBJECT (item), "EsbChoiceSubitems");
+ if (old_subitems) {
+ /* This was connected in set_option() */
+ gtk_signal_disconnect_by_data (GTK_OBJECT (item), old_subitems);
+ free_subitems (old_subitems);
+ gtk_object_set_data (GTK_OBJECT (item), "EsbChoiceSubitems", NULL);
+ }
+
+ if (subitems) {
+ new_subitems = copy_subitems (subitems);
+ gtk_object_set_data (GTK_OBJECT (item), "EsbChoiceSubitems", new_subitems);
+ gtk_signal_connect (GTK_OBJECT (item), "destroy",
+ GTK_SIGNAL_FUNC (option_item_destroy_cb), new_subitems);
+ } else
+ new_subitems = NULL;
+
+ if (search_bar->option_choice == option_id)
+ activate_by_subitems (search_bar, option_id, new_subitems);
+}
+
GtkWidget *
e_search_bar_new (ESearchBarItem *menu_items,
ESearchBarItem *option_items)
@@ -718,6 +778,29 @@ e_search_bar_get_type (void)
}
/**
+ * e_search_bar_set_option_choice:
+ * @search_bar: A search bar.
+ * @id: Identifier of the item to set.
+ *
+ * Sets the active item in the options menu of a search bar.
+ **/
+void
+e_search_bar_set_option_choice (ESearchBar *search_bar, int id)
+{
+ int row;
+
+ g_return_if_fail (search_bar != NULL);
+ g_return_if_fail (E_IS_SEARCH_BAR (search_bar));
+
+ row = find_id (search_bar->option_menu, id, "EsbChoiceId", NULL);
+ g_return_if_fail (row != -1);
+
+ search_bar->option_choice = id;
+ gtk_option_menu_set_history (GTK_OPTION_MENU (search_bar->option), row);
+ emit_query_changed (search_bar);
+}
+
+/**
* e_search_bar_get_option_choice:
* @search_bar: A search bar.
*
@@ -734,6 +817,22 @@ e_search_bar_get_option_choice (ESearchBar *search_bar)
return search_bar->option_choice;
}
+void
+e_search_bar_set_suboption_choice (ESearchBar *search_bar, int id)
+{
+ int row;
+
+ g_return_if_fail (search_bar != NULL);
+ g_return_if_fail (E_IS_SEARCH_BAR (search_bar));
+
+ row = find_id (search_bar->suboption_menu, id, "EsbSubitemId", NULL);
+ g_return_if_fail (row != -1);
+
+ search_bar->suboption_choice = id;
+ gtk_option_menu_set_history (GTK_OPTION_MENU (search_bar->suboption), row);
+ emit_query_changed (search_bar);
+}
+
/**
* e_search_bar_get_suboption_choice:
* @search_bar: A search bar.
@@ -754,6 +853,23 @@ e_search_bar_get_suboption_choice (ESearchBar *search_bar)
}
/**
+ * e_search_bar_set_text:
+ * @search_bar: A search bar.
+ * @text: Text to set in the search bar's entry line.
+ *
+ * Sets the text string inside the entry line of a search bar.
+ **/
+void
+e_search_bar_set_text (ESearchBar *search_bar, const char *text)
+{
+ g_return_if_fail (search_bar != NULL);
+ g_return_if_fail (E_IS_SEARCH_BAR (search_bar));
+
+ e_utf8_gtk_editable_set_text (GTK_EDITABLE (search_bar->entry), text);
+ emit_query_changed (search_bar);
+}
+
+/**
* e_search_bar_get_text:
* @search_bar: A search bar.
*
diff --git a/widgets/misc/e-search-bar.h b/widgets/misc/e-search-bar.h
index 10c4eef8db..0694682e02 100644
--- a/widgets/misc/e-search-bar.h
+++ b/widgets/misc/e-search-bar.h
@@ -47,6 +47,7 @@ extern "C" {
typedef struct {
char *text;
int id;
+ gboolean translate; /* whether to translate the text */
} ESearchBarSubitem;
typedef struct {
@@ -76,7 +77,6 @@ struct _ESearchBar
GtkWidget *dropdown_menu;
GtkWidget *activate_button;
GtkWidget *entry_box;
- GList *subitem_garbage;
guint pending_change;
int option_choice;
@@ -100,6 +100,9 @@ void e_search_bar_set_menu (ESearchBar *search_bar, ESearchBarItem *menu
void e_search_bar_add_menu (ESearchBar *search_bar, ESearchBarItem *menu_item);
void e_search_bar_set_option (ESearchBar *search_bar, ESearchBarItem *option_items);
+void e_search_bar_set_suboption (ESearchBar *search_bar, int option_id,
+ ESearchBarSubitem *subitems);
+
void e_search_bar_construct (ESearchBar *search_bar,
ESearchBarItem *menu_items,
ESearchBarItem *option_items);
@@ -108,8 +111,13 @@ GtkWidget *e_search_bar_new (ESearchBarItem *menu_items,
void e_search_bar_set_menu_sensitive(ESearchBar *search_bar, int id, gboolean state);
+void e_search_bar_set_option_choice (ESearchBar *search_bar, int id);
int e_search_bar_get_option_choice (ESearchBar *search_bar);
+
+void e_search_bar_set_suboption_choice (ESearchBar *search_bar, int id);
int e_search_bar_get_suboption_choice (ESearchBar *search_bar);
+
+void e_search_bar_set_text (ESearchBar *search_bar, const char *text);
char *e_search_bar_get_text (ESearchBar *search_bar);
#ifdef __cplusplus