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/
signature.asc
Description: PGP signature
_______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel