Ticket #11691 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Give feedback during activity saving process

Reported by: martin.langhoff Owned by: erikos
Priority: normal Milestone: 12.1.0
Component: sugar-toolkit Version: not specified
Keywords: Cc: manuq, godiard, dsd
Action Needed: no action Verified: no
Deployments affected: Blocked By:
Blocking:

Description

Steps to repro:

  • On an XO
  • Open Record activity, in a _new_ session
  • Take many pictures, 20 or so
  • Close it
  • In the Name-this-Journal entry change the name (or not), then click the "check" icon to save&close
  • No UI feedbackwhile it's saving! Easy to think it is hung

Ideally, a progress bar. At least a busy cursor spinner is needed here.

Change History

Changed 3 years ago by greenfeld

If this is what I think this is, the Naming Alert has been removed from future versions of Sugar:

 http://wiki.sugarlabs.org/go/Features/Write_to_journal_anytime

However saving progress information might still be nice.

Changed 3 years ago by godiard

We need change this behavior in Record anyway (saving the media on exit).

There are many reports of kids loosing all the production when the activity crash.

Changed 3 years ago by martin.langhoff

That some actitivites will save large / slow files on exit. We do need a spinner there.

Saving media duringthe session in Record seems like a good idea, but is a different bug (and fix) :)

Changed 2 years ago by erikos

  • next_action changed from design to code

A simple spinner for an activity would be:

diff --git a/src/sugar/activity/activity.py b/src/sugar/activity/activity.py
index fdf9827..34e9729 100644
--- a/src/sugar/activity/activity.py
+++ b/src/sugar/activity/activity.py
@@ -897,6 +897,7 @@ class Activity(Window, gtk.Container):
         if not self.can_close():
             return
 
+        self.window.set_cursor(gtk.gdk.Cursor(gtk.gdk.WATCH))
         self.emit('_closing')
 
         if not self._closing:

and for the toolkit-gtk3 respectively: "self.get_window().set_cursor(Gdk.Cursor(Gdk.CursorType.WATCH))"

Ihave applied it here locally and will report back how it feels like.

Changed 2 years ago by erikos

  • summary changed from Name-this-Journal-Entry dialog does not show progress nor wait spinner while saving to Give feedback during activity saving process

I have been tried a few activities and it feels quite good to have the cursor-change feedback during the activity save. Of course this works only for python activities. But Etoys has their own saving mechanics with saving feedback. There is only one more non Python app (Scratch), so we should be ok.

Changed 2 years ago by dsd

  • cc dsd added

Any news here? The trivial patch above looks good to me.

Changed 2 years ago by manuq

I have tested it and also think is useful.

Changed 2 years ago by erikos

Ok, Gary also gave positive feedback about the design and how it feels like. We need to patch sugar-toolkit and sugar-toolkit-gtk3 for this in order to be consistent. I will post patches to sugar-devel.

Changed 2 years ago by dsd

  • next_action changed from review to add to build

Fixed in sugar-toolkit-0.96.2

Changed 2 years ago by pbrobinson

  • next_action changed from add to build to test in build

fix is in os13

Changed 2 years ago by greenfeld

  • status changed from new to closed
  • next_action changed from test in build to no action
  • resolution set to fixed

We now show a busy cursor after the stop button is pressed for an activity in 12.1.0 os13.

Note: See TracTickets for help on using tickets.