From 39be2b7d5389829cdc5ecb7084b1cab12b4b86ea Mon Sep 17 00:00:00 2001 From: David Bordoley Date: Mon, 28 Apr 2003 03:29:20 +0000 Subject: Save the position of windows on the screen, and checks if the position is 2003-04-27 David Bordoley * lib/ephy-state.c: (ephy_state_window_set_position), (ephy_state_window_save_position), (window_configure_event_cb), (window_state_event_cb), (ephy_state_add_window): Save the position of windows on the screen, and checks if the position is off the screen. * src/toolbar.c: (init_bookmarks_toolbar): Pass arguments to egg_toolbars_model_set_flags in the correct order. --- ChangeLog | 14 +++++++++++ lib/ephy-state.c | 77 +++++++++++++++++++++++++++++++++++++++++++++++++++++++- src/toolbar.c | 2 +- 3 files changed, 91 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 9402f28b3..2d9f99c17 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,17 @@ +2003-04-27 David Bordoley + + * lib/ephy-state.c: (ephy_state_window_set_position), + (ephy_state_window_save_position), (window_configure_event_cb), + (window_state_event_cb), (ephy_state_add_window): + + Save the position of windows on the screen, + and checks if the position is off the screen. + + * src/toolbar.c: (init_bookmarks_toolbar): + + Pass arguments to egg_toolbars_model_set_flags in the + correct order. + 2003-04-27 David Bordoley * src/bookmarks/ephy-bookmarks-editor.c: (ephy_bookmarks_editor_construct): diff --git a/lib/ephy-state.c b/lib/ephy-state.c index e89c2db88..44d85909a 100644 --- a/lib/ephy-state.c +++ b/lib/ephy-state.c @@ -29,13 +29,16 @@ #include #define STATES_FILE "states.xml" +#define WINDOW_POSITION_UNSET -1 enum { EPHY_NODE_STATE_PROP_NAME = 2, EPHY_NODE_STATE_PROP_WIDTH = 3, EPHY_NODE_STATE_PROP_HEIGHT = 4, - EPHY_NODE_STATE_PROP_MAXIMIZE = 5 + EPHY_NODE_STATE_PROP_MAXIMIZE = 5, + EPHY_NODE_STATE_PROP_POSITION_X = 6, + EPHY_NODE_STATE_PROP_POSITION_Y = 7 }; static EphyNode *states; @@ -166,6 +169,38 @@ ephy_state_window_set_size (GtkWidget *window, EphyNode *node) } } +static void +ephy_state_window_set_position (GtkWidget *window, EphyNode *node) +{ + GdkScreen *screen; + int x, y; + int screen_width, screen_height; + + g_return_if_fail (GTK_IS_WINDOW (window)); + + /* Setting the default size doesn't work when the window is already showing. */ + g_return_if_fail (!GTK_WIDGET_VISIBLE (window)); + + x = ephy_node_get_property_int (node, EPHY_NODE_STATE_PROP_POSITION_X); + y = ephy_node_get_property_int (node, EPHY_NODE_STATE_PROP_POSITION_Y); + + screen = gtk_window_get_screen (GTK_WINDOW (window)); + screen_width = gdk_screen_get_width (screen); + screen_height = gdk_screen_get_height (screen); + + if ((x >= screen_width) || (y >= screen_height)) + { + x = y = WINDOW_POSITION_UNSET; + } + + /* If the window has a saved position set it, otherwise let the WM do it */ + if ((x != WINDOW_POSITION_UNSET) && (y != WINDOW_POSITION_UNSET)) + { + gtk_window_move (GTK_WINDOW (window), x, y); + } +} + + static void ephy_state_window_save_size (GtkWidget *window, EphyNode *node) { @@ -202,12 +237,34 @@ ephy_state_window_save_size (GtkWidget *window, EphyNode *node) g_value_unset (&value); } +static void +ephy_state_window_save_position (GtkWidget *window, EphyNode *node) +{ + int x,y; + GValue value = { 0, }; + + gtk_window_get_position (GTK_WINDOW (window), &x, &y); + + g_value_init (&value, G_TYPE_INT); + g_value_set_int (&value, x); + ephy_node_set_property (node, EPHY_NODE_STATE_PROP_POSITION_X, + &value); + g_value_unset (&value); + + g_value_init (&value, G_TYPE_INT); + g_value_set_int (&value, y); + ephy_node_set_property (node, EPHY_NODE_STATE_PROP_POSITION_Y, + &value); + g_value_unset (&value); +} + static gboolean window_configure_event_cb (GtkWidget *widget, GdkEventConfigure *event, EphyNode *node) { ephy_state_window_save_size (widget, node); + ephy_state_window_save_position (widget, node); return FALSE; } @@ -217,6 +274,7 @@ window_state_event_cb (GtkWidget *widget, EphyNode *node) { ephy_state_window_save_size (widget, node); + ephy_state_window_save_position (widget, node); return FALSE; } @@ -261,9 +319,26 @@ ephy_state_add_window (GtkWidget *window, ephy_node_set_property (node, EPHY_NODE_STATE_PROP_MAXIMIZE, &value); g_value_unset (&value); + + /* Metacity and presumably any other sane wm won't let + * you drag the titlebar of a window off the screen, so + * we set the inital cordinate to an impossible value (-1,-1) + */ + g_value_init (&value, G_TYPE_INT); + g_value_set_int (&value, WINDOW_POSITION_UNSET); + ephy_node_set_property (node, EPHY_NODE_STATE_PROP_POSITION_X, + &value); + g_value_unset (&value); + + g_value_init (&value, G_TYPE_INT); + g_value_set_int (&value, WINDOW_POSITION_UNSET); + ephy_node_set_property (node, EPHY_NODE_STATE_PROP_POSITION_Y, + &value); + g_value_unset (&value); } ephy_state_window_set_size (window, node); + ephy_state_window_set_position (window, node); g_signal_connect_object (window, "configure_event", G_CALLBACK (window_configure_event_cb), node, 0); diff --git a/src/toolbar.c b/src/toolbar.c index e6c419861..6b366f956 100755 --- a/src/toolbar.c +++ b/src/toolbar.c @@ -344,7 +344,7 @@ init_bookmarks_toolbar (Toolbar *t) t_name); egg_toolbars_model_set_flags (EGG_TOOLBARS_MODEL (model), - i, EGG_TB_MODEL_NOT_REMOVABLE); + EGG_TB_MODEL_NOT_REMOVABLE, i); } } } -- cgit v1.2.3