[Sugar-devel] [PATCH] Display 'Sugar in a window' instead of 'Xephyr on'. Ticket #2285

[Tomeu Vizoso]

On Wed, Sep 8, 2010 at 17:00, Ishan Bansal <ishan at seeta.in> wrote:
> Changed title displayed in the title bar of the sugar-emulator.

Please mention the number of the ticket that this fixes. As Sascha
said before, the commit message would be more useful if you mention to
which string you are changing to.

> ---
>  src/jarabe/util/emulator.py |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/src/jarabe/util/emulator.py b/src/jarabe/util/emulator.py
> index 5a99dbe..07f1569 100644
> --- a/src/jarabe/util/emulator.py
> +++ b/src/jarabe/util/emulator.py
> @@ -20,6 +20,7 @@ import subprocess
>  import sys
>  import time
>  from optparse import OptionParser
> +from gettext import gettext as _
>
>  import gtk
>  import gobject
> @@ -36,7 +37,8 @@ def _run_xephyr(display, dpi, dimensions, fullscreen):
>     cmd = [ 'Xephyr' ]
>     cmd.append(':%d' % display)
>     cmd.append('-ac')
> -
> +    cmd += ['-title',_('Sugar in a window')]

Should be a space after the comma.

> +

You are adding unneeded whitespace here.

This is almost ready to go in.

Regards,

Tomeu

>     screen_size = (gtk.gdk.screen_width(), gtk.gdk.screen_height())
>
>     if (not dimensions) and (fullscreen is None) and \
> --
> 1.7.0.4
>
> _______________________________________________
> Sugar-devel mailing list
> Sugar-devel at lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/sugar-devel
>