[Bugs] #4792 Sugar UNSP: Journal: device palette position is wrong

[Sugar Labs Bugs]

#4792: Journal: device palette position is wrong
----------------------------------------+-------------------------------
  Reporter:  godiard                    |            Owner:
      Type:  defect                     |           Status:  new
  Priority:  Unspecified by Maintainer  |        Milestone:  Unspecified
 Component:  Sugar                      |          Version:  Unspecified
  Severity:  Unspecified                |       Resolution:
  Keywords:  F20                        |  Distribution/OS:  Unspecified
Bug Status:  Unconfirmed                |
----------------------------------------+-------------------------------

Comment (by SAMdroid):

 This seems to be a bug with the estimation of the palette size.  The
 secondary text should be up to 3 lines but it only ever shows as 1.

 I have isolated the code to the GTK 3.10+ branch of
 sugar3.graphics.palette.Palette.set_secondary_text.  This stops the issue
 from occurring:


 {{{
 diff --git a/src/sugar3/graphics/palette.py
 b/src/sugar3/graphics/palette.py
 index 5a0bfea..0804745 100644
 --- a/src/sugar3/graphics/palette.py
 +++ b/src/sugar3/graphics/palette.py
 @@ -291,7 +291,7 @@ class Palette(PaletteWindow):
              label = label.replace('\n', ' ')
              label = label.replace('\r', ' ')

 -            if hasattr(self._secondary_label, 'set_lines'):
 +            if hasattr(self._secondary_label, 'set_lines') and False:
                  self._secondary_label.set_max_width_chars(
                      style.MENU_WIDTH_CHARS)
                  self._secondary_label.set_line_wrap(True)

 }}}

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