[sugar] [PATCH] Fixing timestamp issue on wnck.Window.activate()

Marco Pesenti Gritti mpgritti at gmail.com
Wed Jun 4 13:49:35 EDT 2008


On Wed, Jun 4, 2008 at 7:37 PM, Sayamindu Dasgupta <sayamindu at gmail.com> wrote:
> On Wed, Jun 4, 2008 at 10:38 PM, Marco Pesenti Gritti
> <mpgritti at gmail.com> wrote:
>>     def present(self):
>> -        # wnck.Window.activate() expects a timestamp, but we don't
>> -        # always have one, and libwnck will complain if we pass "0",
>> -        # and matchbox doesn't look at the timestamp anyway. So we
>> -        # just always pass "1".
>> -        self._window.activate(1)
>> +        self._window.activate(gtk.get_current_event_time())
>>
>> Since you can't be sure of where present() will be called from (an
>> event or not), I'd rather pass in a timestamp to method.
>
> Hmm - but passing 1 does not seem to work in case of Metacity, The
> documentation for gtk.get_current_event_time() suggests that 0L is
> returned if there is no current event.

Sorry I was not clear enough... I was proposing to change the method
to present(self, timestamp).

Though I just looked at how gtk handles it and they seem to default to
current time, providing a different method for the special cases
(present_with_time).

So in summary I think your patch is good except for the 80 line
nitpick. Please fix that and check it in...

Thanks!!

Marco


More information about the Sugar mailing list