On Thu, Feb 19, 2009 at 12:22:50PM +0100, Bernie Innocenti wrote:

      if dimensions is not None:
          cmd.append('-screen')
          cmd.append(dimensions)
- elif gtk.gdk.screen_width() < 1200 or gtk.gdk.screen_height() < 900:
+    elif gtk.gdk.screen_width() < DEFAULT_WIDTH or \
+            gtk.gdk.screen_height() < DEFAULT_HEIGHT:
          cmd.append('-fullscreen')
      else:
          cmd.append('-screen')
-        cmd.append('%dx%d' % (1200, 900))
+        cmd.append('%dx%d' % (DEFAULT_WIDTH, DEFAULT_HEIGHT))
Is it on purpose that non-default dimensions can be set (see first three lines) but for determining whether fullscreen will be used the default dimensions get examined instead? Yes, that part isn't changed by the patch and I've wondered about it before, but not bothered to ask. :)

CU Sascha

--
http://sascha.silbe.org/
http://www.infra-silbe.de/

Attachment: signature.asc
Description: Digital signature

_______________________________________________
Sugar-devel mailing list
[email protected]
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to