[Pidgin] #4461: Unique window positions for new windows

Pidgin trac at pidgin.im
Sun Feb 3 19:14:58 EST 2008


#4461: Unique window positions for new windows
------------------------------+---------------------------------------------
  Reporter:  vboxw1           |       Owner:  datallah               
      Type:  enhancement      |      Status:  new                    
  Priority:  minor            |   Milestone:                         
 Component:  winpidgin (gtk)  |     Version:  2.3.1                  
Resolution:                   |    Keywords:  Position offset windows
   Pending:  0                |  
------------------------------+---------------------------------------------
Comment (by lex):

 looking at pidgin-2.3.1/pidgin/gtkconv.c:
 {{{
 PidginWindow *
 pidgin_conv_window_new()
 {
         PidginWindow *win;
         GtkPositionType pos;
         GtkWidget *testidea;
         GtkWidget *menubar;
 #if GTK_CHECK_VERSION(2,6,0)
         GdkModifierType state;
 #endif

         win = g_malloc0(sizeof(PidginWindow));

         window_list = g_list_append(window_list, win);

         /* Create the window. */
         win->window = pidgin_create_window(NULL, 0, "conversation", TRUE);
 #if GTK_CHECK_VERSION(2,6,0)
         if (!gtk_get_current_event_state(&state))
                 gtk_window_set_focus_on_map(GTK_WINDOW(win->window),
 FALSE);
 #endif
         pidgin_conv_restore_position(win);
 ...
 }}}

 {{{
 static void
 pidgin_conv_restore_position(PidginWindow *win) {
         pidgin_conv_set_position_size(win,
                 purple_prefs_get_int(PIDGIN_PREFS_ROOT
 "/conversations/im/x"),
                 purple_prefs_get_int(PIDGIN_PREFS_ROOT
 "/conversations/im/y"),
                 purple_prefs_get_int(PIDGIN_PREFS_ROOT
 "/conversations/im/width"),
                 purple_prefs_get_int(PIDGIN_PREFS_ROOT
 "/conversations/im/height"));
 }
 }}}

 ...and pidgin_conv_set_position_size also doesn't have any code to check
 the platform either, so position-storing is definitely happening on all
 platforms.  Should I open a new issue?

-- 
Ticket URL: <http://developer.pidgin.im/ticket/4461#comment:4>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list