[Bugs] #3569 sugar-artwork NORM: Icons inside text entries are not centered vertically

[Sugar Labs Bugs]

#3569: Icons inside text entries are not centered vertically
------------------------------+---------------------------------------------
    Reporter:  tonyforster    |          Owner:  erikos                
        Type:  defect         |         Status:  new                   
    Priority:  Normal         |      Milestone:  0.98                  
   Component:  sugar-artwork  |        Version:  Unspecified           
    Severity:  Unspecified    |       Keywords:  screenshot, regression
Distribution:  Unspecified    |   Status_field:  Unconfirmed           
------------------------------+---------------------------------------------

Comment(by manuq):

 So commenting out set_fonts() in sugar-session the icon looks right.  This
 shows that the icon inside the entry at the position
 Gtk.EntryIconPosition.SECONDARY depends on the font.  The attached
 animation also shows that the other icon of position
 Gtk.EntryIconPosition.PRIMARY is not affected by the font.

 I have added debug prints in the XO to know the font values before and
 after and they are:

 {{{
 TEST current font: 'Sans 10'
 TEST new font: 'Sans Serif 7.000000'
 }}}

 {{{
 --- a/bin/sugar-session
 +++ b/bin/sugar-session
 @@ -242,7 +242,11 @@ def set_fonts():
      face = client.get_string('/desktop/sugar/font/default_face')
      size = client.get_float('/desktop/sugar/font/default_size')
      settings = Gtk.Settings.get_default()
 +    curfont = settings.get_property("gtk-font-name")
 +    logging.debug("TEST current font: %r", curfont)
      settings.set_property("gtk-font-name", "%s %f" % (face, size))
 +    newfont = settings.get_property("gtk-font-name")
 +    logging.debug("TEST new font: %r", newfont)
 }}}

 The difference that humitos and I noticed from running a simple test
 script in a sugar shell was because that script is not setting the font
 like sugar-session or the Activity class constructor do.

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/3569#comment:14>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system