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

[Zarro Boogs per Child]

#7823: 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.

 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.

 We should make the Cancel button in the toolbar insensitive as well,
 forcing the action to be selected from the alert instead.  I'm OK with
 making the alert button behave slightly differently, keeping the module
 visible while reverting the changes, allowing the user to then adjust a
 subset of the previously changed preferences, or explicitly leave the
 module via the Cancel button in the toolbar.

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