[nautilus] window: remove unused code for temporary search/location bars
- From: Cosimo Cecchi <cosimoc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [nautilus] window: remove unused code for temporary search/location bars
- Date: Sat, 14 Jul 2012 21:23:10 +0000 (UTC)
commit 75fcef36d94bf49e9b493af461c5c643334a8cf0
Author: Cosimo Cecchi <cosimoc gnome org>
Date: Sat Jul 14 17:22:09 2012 -0400
window: remove unused code for temporary search/location bars
Since we always have a toolbar, we don't need this logic anymore.
src/nautilus-toolbar.c | 30 --------------------------
src/nautilus-window-private.h | 1 -
src/nautilus-window.c | 47 +----------------------------------------
3 files changed, 1 insertions(+), 77 deletions(-)
---
diff --git a/src/nautilus-toolbar.c b/src/nautilus-toolbar.c
index db04293..1492faa 100644
--- a/src/nautilus-toolbar.c
+++ b/src/nautilus-toolbar.c
@@ -48,7 +48,6 @@ struct _NautilusToolbarPriv {
GtkToolItem *back_forward;
- gboolean show_main_bar;
gboolean show_location_entry;
};
@@ -56,7 +55,6 @@ enum {
PROP_ACTION_GROUP = 1,
PROP_UI_MANAGER,
PROP_SHOW_LOCATION_ENTRY,
- PROP_SHOW_MAIN_BAR,
NUM_PROPERTIES
};
@@ -72,9 +70,6 @@ toolbar_update_appearance (NautilusToolbar *self)
show_location_entry = self->priv->show_location_entry ||
g_settings_get_boolean (nautilus_preferences, NAUTILUS_PREFERENCES_ALWAYS_USE_LOCATION_ENTRY);
- gtk_widget_set_visible (self->priv->toolbar,
- self->priv->show_main_bar);
-
gtk_widget_set_visible (self->priv->location_bar,
show_location_entry);
gtk_widget_set_visible (self->priv->path_bar,
@@ -227,7 +222,6 @@ nautilus_toolbar_init (NautilusToolbar *self)
{
self->priv = G_TYPE_INSTANCE_GET_PRIVATE (self, NAUTILUS_TYPE_TOOLBAR,
NautilusToolbarPriv);
- self->priv->show_main_bar = TRUE;
}
static void
@@ -242,9 +236,6 @@ nautilus_toolbar_get_property (GObject *object,
case PROP_SHOW_LOCATION_ENTRY:
g_value_set_boolean (value, self->priv->show_location_entry);
break;
- case PROP_SHOW_MAIN_BAR:
- g_value_set_boolean (value, self->priv->show_main_bar);
- break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec);
break;
@@ -269,9 +260,6 @@ nautilus_toolbar_set_property (GObject *object,
case PROP_SHOW_LOCATION_ENTRY:
nautilus_toolbar_set_show_location_entry (self, g_value_get_boolean (value));
break;
- case PROP_SHOW_MAIN_BAR:
- nautilus_toolbar_set_show_main_bar (self, g_value_get_boolean (value));
- break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, property_id, pspec);
break;
@@ -321,12 +309,6 @@ nautilus_toolbar_class_init (NautilusToolbarClass *klass)
"Whether to show the location entry instead of the pathbar",
FALSE,
G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS);
- properties[PROP_SHOW_MAIN_BAR] =
- g_param_spec_boolean ("show-main-bar",
- "Whether to show the main bar",
- "Whether to show the main toolbar",
- TRUE,
- G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS);
g_type_class_add_private (klass, sizeof (NautilusToolbarClass));
g_object_class_install_properties (oclass, NUM_PROPERTIES, properties);
@@ -356,18 +338,6 @@ nautilus_toolbar_get_location_bar (NautilusToolbar *self)
}
void
-nautilus_toolbar_set_show_main_bar (NautilusToolbar *self,
- gboolean show_main_bar)
-{
- if (show_main_bar != self->priv->show_main_bar) {
- self->priv->show_main_bar = show_main_bar;
- toolbar_update_appearance (self);
-
- g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_SHOW_MAIN_BAR]);
- }
-}
-
-void
nautilus_toolbar_set_show_location_entry (NautilusToolbar *self,
gboolean show_location_entry)
{
diff --git a/src/nautilus-window-private.h b/src/nautilus-window-private.h
index 6c42bd4..4ebec16 100644
--- a/src/nautilus-window-private.h
+++ b/src/nautilus-window-private.h
@@ -87,7 +87,6 @@ struct NautilusWindowDetails
GtkWidget *toolbar;
GtkActionGroup *toolbar_action_group;
gboolean temporary_navigation_bar;
- gboolean temporary_search_bar;
/* focus widget before the location bar has been shown temporarily */
GtkWidget *last_focus_widget;
diff --git a/src/nautilus-window.c b/src/nautilus-window.c
index 12d2de6..f346294 100644
--- a/src/nautilus-window.c
+++ b/src/nautilus-window.c
@@ -197,32 +197,6 @@ bookmark_list_get_uri_index (GList *list, GFile *location)
}
static void
-nautilus_window_hide_temporary_bars (NautilusWindow *window)
-{
- NautilusWindowSlot *slot;
- NautilusDirectory *directory;
-
- slot = window->details->active_slot;
-
- if (window->details->temporary_navigation_bar) {
- directory = nautilus_directory_get (slot->location);
-
- window->details->temporary_navigation_bar = FALSE;
-
- /* if we're in a search directory, hide the main bar, and show the search
- * bar again; otherwise, just hide the whole toolbar.
- */
- if (NAUTILUS_IS_SEARCH_DIRECTORY (directory)) {
- nautilus_toolbar_set_show_main_bar (NAUTILUS_TOOLBAR (window->details->toolbar), FALSE);
- } else {
- gtk_widget_hide (window->details->toolbar);
- }
-
- nautilus_directory_unref (directory);
- }
-}
-
-static void
unset_focus_widget (NautilusWindow *window)
{
if (window->details->last_focus_widget != NULL) {
@@ -232,27 +206,13 @@ unset_focus_widget (NautilusWindow *window)
}
}
-static gboolean
-widget_is_in_temporary_bars (GtkWidget *widget,
- NautilusWindow *window)
-{
- gboolean res = FALSE;
-
- if ((gtk_widget_get_ancestor (widget, NAUTILUS_TYPE_LOCATION_BAR) != NULL &&
- window->details->temporary_navigation_bar))
- res = TRUE;
-
- return res;
-}
-
static void
remember_focus_widget (NautilusWindow *window)
{
GtkWidget *focus_widget;
focus_widget = gtk_window_get_focus (GTK_WINDOW (window));
- if (focus_widget != NULL &&
- !widget_is_in_temporary_bars (focus_widget, window)) {
+ if (focus_widget != NULL) {
unset_focus_widget (window);
window->details->last_focus_widget = focus_widget;
@@ -289,7 +249,6 @@ navigation_bar_cancel_callback (GtkWidget *widget,
{
nautilus_toolbar_set_show_location_entry (NAUTILUS_TOOLBAR (window->details->toolbar), FALSE);
- nautilus_window_hide_temporary_bars (window);
restore_focus_widget (window);
}
@@ -299,7 +258,6 @@ navigation_bar_location_changed_callback (GtkWidget *widget,
NautilusWindow *window)
{
nautilus_toolbar_set_show_location_entry (NAUTILUS_TOOLBAR (window->details->toolbar), FALSE);
- nautilus_window_hide_temporary_bars (window);
restore_focus_widget (window);
@@ -780,8 +738,6 @@ nautilus_window_sync_location_widgets (NautilusWindow *window)
slot = window->details->active_slot;
- nautilus_window_hide_temporary_bars (window);
-
/* Change the location bar and path bar to match the current location. */
if (slot->location != NULL) {
char *uri;
@@ -819,7 +775,6 @@ nautilus_window_ensure_location_bar (NautilusWindow *window)
remember_focus_widget (window);
- nautilus_toolbar_set_show_main_bar (NAUTILUS_TOOLBAR (window->details->toolbar), TRUE);
nautilus_toolbar_set_show_location_entry (NAUTILUS_TOOLBAR (window->details->toolbar), TRUE);
location_bar = nautilus_toolbar_get_location_bar (NAUTILUS_TOOLBAR (window->details->toolbar));
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]