Jonas Smedegaard wrote: >> +DEFAULT_WIDTH = 1024 >> +DEFAULT_HEIGHT = 768 > > How about adjusting that patch to keep old values by default, but > provide commandline arguments and perhaps let environment variables > overrid default too)? > > That would avoid surprises, while still allow testing alternate sizes > without hacking the code.
We already have a command line argument to change the default size, and few activity authors seem to test with anything but the default. The intention of this change is to surprise them a little. We might even change it every once in a while to catch those who have hard-coded or special-cased the new default in their code. -- // Bernie Innocenti - http://www.codewiz.org/ \X/ Sugar Labs - http://www.sugarlabs.org/ _______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
