#7881 HIGH 8.2.0 (: Accelerator labels don't show up for most toolbar buttons

[Zarro Boogs per Child]

#7881: Accelerator labels don't show up for most toolbar buttons
----------------------+-----------------------------------------------------
   Reporter:  Eben    |       Owner:  Eben                             
       Type:  defect  |      Status:  new                              
   Priority:  high    |   Milestone:  8.2.0 (was Update.2)             
  Component:  sugar   |     Version:  Development build as of this date
 Resolution:          |    Keywords:  8.2.0:? r+                       
Next_action:  review  |    Verified:  0                                
  Blockedby:          |    Blocking:                                   
----------------------+-----------------------------------------------------
Changes (by tomeu):

  * keywords:  8.2.0:? r? => 8.2.0:? r+


Comment:

 {{{
 for hid in self._invoker_hids:
     self._invoker.disconnect(hid)
 self._invoker_hids = []
 }}}

 I would prefer in the following way, so _invoker_hids isn't left with
 invalid data if an exception is raised inside disconnect():

 {{{
 for hid in self._invoker_hids[:]:
     self._invoker.disconnect(hid)
     self._invoker_hids.remove(hid)
 }}}

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