[Sugar-devel] [PATCH sugar-artwork] Theme GtkSpinButton buttons SL #3406

[Simon Schampijer]

Thanks Manuel for the patch!

I commented about the design in: 
http://bugs.sugarlabs.org/ticket/3406#comment:7

Regards,
    Simon

On 05/30/2012 08:08 PM, Manuel Quiñones wrote:
> This patch sets border radius to the buttons, so the layout is like
> (value [-][+)) .
>
> Also removes two unneeded variables.
>
> Signed-off-by: Manuel Quiñones<manuq at laptop.org>
> ---
>   gtk3/theme/gtk-widgets.css.em |   10 ++++++----
>   1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/gtk3/theme/gtk-widgets.css.em b/gtk3/theme/gtk-widgets.css.em
> index 9671e29..d268053 100644
> --- a/gtk3/theme/gtk-widgets.css.em
> +++ b/gtk3/theme/gtk-widgets.css.em
> @@ -173,19 +173,21 @@ icon_large = icon_base * 5
>
>   ${ spin_ythickness = my_ceil(3*subcell_size - font_height) }
>   ${ spin_xthickness = subcell_size }
> -${ spin_btn_ythickness = spin_ythickness }
> -${ spin_btn_xthickness = subcell_size * 2 }
>
>   .spinbutton.button,
>   .spinbutton.button:focused,
>   .spinbutton.button:active {
>       padding: $(spin_ythickness)px $(spin_xthickness)px;
> -    border-color: @button_grey;
> -    border-width: $(thickness)px;
> +    border-radius: 0px;
> +    border-width: 0px;
>       background-color: @button_grey;
>       color: @white;
>   }
>
> +.spinbutton.button:last-child {
> +    border-radius: 0px $(subcell_size)px $(subcell_size)px 0px;
> +}
> +
>   .spinbutton.button:active {
>       background-color: @white;
>       color: @black;