Excerpts from Ishan Bansal's message of Mon Sep 20 17:36:03 +0200 2010:

> The sections in control panel should activate busy cursor so that user could 
> be given a impression that their request is in progress.

Like for the other patch, please wrap lines so they fit into 80 columns
(70-75 for the subject).

[src/jarabe/controlpanel/gui.py]
> @@ -214,11 +214,14 @@ class ControlPanel(gtk.Window):
>                           globals(), locals(), ['model'])
>          model = ModelWrapper(mod)
>  

> +        self.get_window().set_cursor(gtk.gdk.Cursor(gtk.gdk.WATCH)) 
> +        
>          self._section_view = view_class(model,
>                                          self._options[option]['alerts'])
>  
>          self._set_canvas(self._section_view)
>          self._section_view.show()
> +        self.get_window().set_cursor(None)

The cursor shape is global state, so it should be reset in a finally:
clause with everything between the two set_cursor() calls in the try:
block.

Sascha

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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to