[Sugar-devel] [PATCH] Color Palette: get back horizontal padding - SL #4325
Simon Schampijer
simon at schampijer.de
Wed Apr 24 10:38:58 EDT 2013
On 04/19/2013 03:51 PM, Manuel Quiñones wrote:
> 2013/4/18 Simon Schampijer <simon at schampijer.de>:
>> Hi Manuel,
>>
>> your patch does fix the Write-Color-Palette.
>>
>> Which is the Abacus-Palette, the custom one? Is Walter aware of this fix and
>> would remove his workaround (at least in master)?
>
> Yes, the Abacus palette is the custom one. I'm prepared to let Walter
> know about this change at the moment its commited so he can remove the
> workaround. Gonzalo workarounded Paint too.
>
>> Code looks good, please push, as well to 0.98.
>
> I have sent a new patch which adds padding as optional parameter. I
> think it is better API. What do you think?
>
> --
> .. manuq ..
The API is good, but as you add API you can not push as-is to the 0.98
branch. You can push the first one there.
Simon
More information about the Sugar-devel
mailing list