make human logic safe resizements, moves of windows

src/groupchat_window.py

@@ -90,9 +90,9 @@ class GroupchatWindow(chat.Chat):
 
 		# get size and position from config
 		if gajim.config.get('saveposition'):
-			self.window.move(gajim.config.get('gc-x-position'),
-					gajim.config.get('gc-y-position'))
-			self.window.resize(gajim.config.get('gc-width'),
+			gtkgui_helpers.move_window(self.window, gajim.config.get('gc-x-position'),
+				gajim.config.get('gc-y-position'))
+			gtkgui_helpers.resize(self.window, gajim.config.get('gc-width'),
 					gajim.config.get('gc-height'))
 		self.window.show_all()
 

src/gtkgui_helpers.py

@@ -27,6 +27,10 @@ _ = i18n._
 from common import gajim
 from common import helpers
 
+screen = gtk.gdk.Screen()
+screen_w, screen_h = screen.get_width(), screen.get_height()
+del screen
+
 def get_default_font():
 	''' Get the desktop setting for application font
 	first check for GNOME, then XFCE and last KDE
@@ -161,3 +165,19 @@ def autodetect_browser_mailer():
 			gajim.config.set('openwith', 'kfmclient exec')
 		else:
 			gajim.config.set('openwith', 'custom')
+
+def move_window(window, x, y):
+	''' moves the window but also checks if out of screen '''
+	if x < 0:
+		x = 0
+	if y < 0:
+		y = 0
+	window.move(x, y)
+
+def resize_window(window, w, h):
+	''' resizes window but also checks if huge window or negative values '''
+	if w > screen_w:
+		w = screen_w
+	if h > screen_h:
+		h = screen_h
+	window.resize(abs(w), abs(h))

src/roster_window.py

@@ -2040,14 +2040,9 @@ _('If "%s" accepts this request you will know his status.') %jid)
 		# no need of this variable
 		self.have_new_message_accel = False # Is the "Ctrl+N" shown ?
 		if gajim.config.get('saveposition'):
-			x = gajim.config.get('roster_x-position')
-			y = gajim.config.get('roster_y-position')
-			if x < 0:
-				x = 0
-			if y < 0:
-				y = 0
-			self.window.move(x, y)
-			self.window.resize(gajim.config.get('roster_width'),
+			gtkgui_helpers.move_window(self.window, gajim.config.get('roster_x-position'),
+				gajim.config.get('roster_y-position'))
+			gtkgui_helpers.resize_window(self.window, gajim.config.get('roster_width'),
 				gajim.config.get('roster_height'))
 
 		self.popups_notification_height = 0

src/tabbed_chat_window.py

@@ -72,9 +72,9 @@ class TabbedChatWindow(chat.Chat):
 
 		if gajim.config.get('saveposition'):
 			# get window position and size from config
-			self.window.move(gajim.config.get('chat-x-position'),
-					gajim.config.get('chat-y-position'))
-			self.window.resize(gajim.config.get('chat-width'),
+			gtkgui_helpers.move_window(self.window, gajim.config.get('chat-x-position'),
+				gajim.config.get('chat-y-position'))
+			gtkgui_helpers.resize(self.window, gajim.config.get('chat-width'),
 					gajim.config.get('chat-height'))
 
 		# gtk+ doesn't make use of the motion notify on gtkwindow by default