#7987 NORM 8.2.0 (: Home view XO icon palette for Control Panel has wrong icon

[Zarro Boogs per Child]

#7987: Home view XO icon palette for Control Panel has wrong icon
---------------------------+------------------------------------------------
   Reporter:  garycmartin  |       Owner:  Eben                             
       Type:  defect       |      Status:  new                              
   Priority:  normal       |   Milestone:  8.2.0 (was Update.2)             
  Component:  sugar        |     Version:  Development build as of this date
 Resolution:               |    Keywords:  8.2.0:polish 8.2.0:+ r+          
Next_action:  review       |    Verified:  0                                
  Blockedby:               |    Blocking:                                   
---------------------------+------------------------------------------------

Comment(by Eben):

 Here's the updated patch, which uses a dedicated settings icon and also
 changes the name to "Settings".  We (Sugar team) feel it's important to
 provide the correct name/icon the first time this feature is released,
 which is why we feel comfortable requesting the string freeze break, which
 has already been approved.

 {{{
 -        item = MenuItem(_('Control Panel'))
 -
 -        icon = Icon(icon_name='computer-xo',
 icon_size=gtk.ICON_SIZE_MENU,
 -                xo_color=self._profile.color)
 -        item.set_image(icon)
 -        icon.show()
 -
 +        item = MenuItem(_('Settings'), 'preferences-system')
          item.connect('activate', self.__controlpanel_activate_cb)
          palette.menu.append(item)
          item.show()

 }}}

 In a moment, I'll attach a diff for artwork that's needed to add this new
 icon.  I'd like that reviewed as well, since it required a few more lines
 of code than usual.

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