#2630 HIGH Opportu: Not enough feedback for unmount failure

Zarro Boogs per Child bugtracker at laptop.org
Wed Dec 5 11:12:26 EST 2007


#2630: Not enough feedback for unmount failure
-------------------------------+--------------------------------------------
  Reporter:  Zack              |       Owner:  rwh        
      Type:  enhancement       |      Status:  assigned   
  Priority:  high              |   Milestone:  Opportunity
 Component:  journal-activity  |     Version:             
Resolution:                    |    Keywords:             
  Verified:  0                 |  
-------------------------------+--------------------------------------------

Comment(by rwh):

 I attached a patch to implement async unmounting and Volume state handling
 in the UI. See also my personal git tree.

 The Volume class has a state added, and emits a signal when this is
 changed. The VolumesManager forwards this signal, and the VolumesToolbar
 connect()s to that to modify the icon accordingly. Visual feedback is
 currently by badges, but could be the suggestion of Even as well with
 changing colors. Btw, Eben: the warning badge is not in the current
 artwork; that might be useful for other purposes as well.

 I distinguish several different states:
 *STATE_REMOVE: Not plugged in
 *STATE_INSERTED: Plugged in, not mounted in system or DS
 *STATE_PRESENT: mounted in system
 *STATE_MOUNTED: mounted in system and DS
 *STATE_BUSY: changing state (from STATE_MOUNTED to STATE_REMOVED for
 example)
 *STATE_WARNING, STATE_ERROR: something went wrong. Lasts 10secs [now, I
 suggest 20s or so] and gets replaced by a more appropriate final state.

 I think this gives us the flexibility to give the user all the feedback he
 needs. Note that Tomeu suggested that in general we should never be in the
 state where the system has the device mounted and the DS is unmounted;
 this should only happen if the user accesses the device outside of the DS.

 We do not need this many visual icon states, but can map several states to
 the same icon + badge.

-- 
Ticket URL: <http://dev.laptop.org/ticket/2630#comment:22>
One Laptop Per Child <http://dev.laptop.org>
OLPC bug tracking system



More information about the Bugs mailing list