[Dextrose] [Sugar-devel] Fwd: [PATCH] Changed outline of checkbox from grey to black when activated. Ticket #305

[Frederick Grose]

On Wed, Oct 20, 2010 at 9:15 AM, Ishan Bansal <ishan at seeta.in> wrote:

>
> Hi
>
> I have submitted the patch for the ticket #305.
>
> Wish if you could review it and provide me suggestions on any improvement
> required.
>
> Regards
>
> ishan
>
> ---------- Forwarded message ----------
> From: Ishan Bansal <ishan at seeta.in>
> Date: Fri, Sep 10, 2010 at 12:07 AM
> Subject: [PATCH] Changed outline of checkbox from grey to black when
> activated. Ticket #305
> To: sugar-devel at lists.sugarlabs.org
>
>
> Changed outline of checkbox from grey to black so that it does not
> disappears into the grey highlight of the mouseover selection. (Ticket #305)
> ---
>  gtk/theme/gtkrc.em |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/gtk/theme/gtkrc.em b/gtk/theme/gtkrc.em
> index 69dccd9..3c0d7f6 100644
> --- a/gtk/theme/gtkrc.em
> +++ b/gtk/theme/gtkrc.em
> @@ -48,6 +48,7 @@ button_grey = '"#808080"'
>  selection_grey = '"#A6A6A6"'
>  panel_grey = '"#C0C0C0"'
>  text_field_grey = '"#E5E5E5"'
> +text_field_black = '"#000000"'
>  white = '"#FFFFFF"'
>
>
> @@ -630,7 +631,7 @@ style "checkbutton"
>  {
>     base[NORMAL]      = $white
>     base[PRELIGHT]    = $white
> -    base[ACTIVE]      = $text_field_grey
> +    base[ACTIVE]      = $text_field_black
>
>     text[PRELIGHT]    = $toolbar_grey
>     text[NORMAL]      = $toolbar_grey
> --
> 1.7.0.4
>

Considering comment #4, http://bugs.sugarlabs.org/ticket/305#comment:4,
could you please recommend menu in a current version of Sugar where this can
be tested?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/dextrose/attachments/20101020/007ac32f/attachment.html>