Re: [Sugar-devel] [PATCH shell 2/2] Port Volume palette that has mixed content to new API

2012-09-20 Thread Simon Schampijer
That one looks good as well, please push. Simon On 09/20/2012 07:00 AM, Manuel Quiñones wrote: This fixes VolumePalette that is used in the Journal, and in the volume device icon. Had to set the content box (the one passed to set_content) a property of the palette. Otherwise the Show

Re: [Sugar-devel] [PATCH shell 2/2] Port Volume palette that has mixed content to new API

2012-09-20 Thread Ignacio Rodríguez
... How odd. Do tried adding all end? (Pack_start)? 2012/9/20, Simon Schampijer si...@schampijer.de: That one looks good as well, please push. Simon On 09/20/2012 07:00 AM, Manuel Quiñones wrote: This fixes VolumePalette that is used in the Journal, and in the volume device icon. Had to

Re: [Sugar-devel] [PATCH shell 2/2] Port Volume palette that has mixed content to new API

2012-09-20 Thread Manuel Quiñones
2012/9/20 Simon Schampijer si...@schampijer.de: That one looks good as well, please push. Pushed fixing the typo, thanks, Simon On 09/20/2012 07:00 AM, Manuel Quiñones wrote: This fixes VolumePalette that is used in the Journal, and in the volume device icon. Had to set the content

[Sugar-devel] [PATCH shell 2/2] Port Volume palette that has mixed content to new API

2012-09-19 Thread Manuel Quiñones
This fixes VolumePalette that is used in the Journal, and in the volume device icon. Had to set the content box (the one passed to set_content) a property of the palette. Otherwise the Show Contents can't be added from the device icon. Previusly the palette.menu attribute was used for that.