Hello team, I have attached the screen shot for the check box without making the changes present in the patch. I have made a new window to test the working of the checkbox. The checkbox seems to be working fine. Please find the two screenshots of the checkbox (both selected and unselected) attached. Regards Shanjit Singh Jajmann <div class="gmail_quote">On Wed, Oct 20, 2010 at 9:45 PM, Ishan Bansal <<a href="mailto:ishan@seeta.in">ishan@seeta.in</a>> wrote: <blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> 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 <div class="gmail_quote"> ---------- Forwarded message ---------- From: Ishan Bansal <<a href="mailto:ishan@seeta.in" target="_blank">ishan@seeta.in</a>> Date: Fri, Sep 10, 2010 at 12:07 AM Subject: [PATCH] Changed outline of checkbox from grey to black when activated. Ticket #305 To: <a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank">sugar-devel@lists.sugarlabs.org</a> 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 </div> _______________________________________________ Dextrose mailing list <a href="mailto:Dextrose@lists.sugarlabs.org">Dextrose@lists.sugarlabs.org</a> <a href="http://lists.sugarlabs.org/listinfo/dextrose" target="_blank">http://lists.sugarlabs.org/listinfo/dextrose</a> </blockquote></div>