#7822 HIGH 8.2.0 (: Non-modal alerts in CP remain when they shouldn't

[Zarro Boogs per Child]

#7822: Non-modal alerts in CP remain when they shouldn't
--------------------+-------------------------------------------------------
 Reporter:  Eben    |         Owner:  erikos                           
     Type:  defect  |        Status:  new                              
 Priority:  high    |     Milestone:  8.2.0 (was Update.2)             
Component:  sugar   |       Version:  Development build as of this date
 Keywords:          |   Next_action:  code                             
 Verified:  0       |     Blockedby:                                   
 Blocking:          |  
--------------------+-------------------------------------------------------
 The CP offers non-modal alerts when changes require a restart to take
 effect.  When this alert appears, the Accept button is (correctly) made
 insensitive, since the alert is an intermediary confirming the accept
 action.  However, the cancel button remains available.  There are 2
 problems:

  1. The Cancel button in the alert reverts the changes but remains in the
 module, while the Cancel button in the toolbar reverts changes and returns
 to the module selection screen.  This is inconsistent.

  2. When pressing the Cancel button in the toolbar, we are returned to the
 module selection screen, but the non-modal alert which referred to the
 previously selected module remains visible.  This is ''very'' confusing.

 There are two solutions which solve both of these problems:

  1. Make the Cancel button in the toolbar insensitive as well, forcing the
 action to be selected from the alert instead.

  2. Make the cancel button in the alert return to the module selection
 screen just like that in the toolbar.

  3. (Or both 1 and 2)

 I lean towards option 1, keeping the module visible, but reverting the
 changes, allowing the user to then adjust a subset of the previously
 changed preferences, or explicitly leave the module.

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