#7643 NORM 9.1.0: Control Panel accept/cancel buttons are confusing

[Zarro Boogs per Child]

#7643: Control Panel accept/cancel buttons are confusing
----------------------+-----------------------------------------------------
   Reporter:  Eben    |       Owner:  erikos                           
       Type:  defect  |      Status:  new                              
   Priority:  normal  |   Milestone:  9.1.0                            
  Component:  sugar   |     Version:  Development build as of this date
 Resolution:          |    Keywords:  8.2.0:? r?                       
Next_action:  review  |    Verified:  0                                
  Blockedby:          |    Blocking:                                   
----------------------+-----------------------------------------------------
Changes (by Eben):

  * keywords:  => 8.2.0:? r?
  * next_action:  communicate => review


Comment:

 I've attached a patch which makes the buttons more intuitive by setting
 their sensitivity dynamically.  This patch reveals one small problem,
 which is that no state from the module is stored when leaving/returning.
 This means that, if you make changes to a given module, leave by selecting
 "later" when prompted for restart, and then return, the modified and
 needs_restart variables get reset to False, even though both are actually
 true.  This results in an insensitive accept button, even though there are
 changes.  It's arguable whether or not we wish to prompt again when
 leaving the module a second time.

 I'd still personally like to see this make 8.2.0, since I think it does a
 lot to make the CP more natural to use.

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