kicad-developers team mailing list archive

[Lorenzo Marcantonio <l.marcantonio@xxxxxxxxxxxx>]

A minipatch fixing a bug I found while investigating a (probably
invalid) plotting bug.

The eeschema color configuration dialog didn't contain the 
'sheet label' color option (and got set to 0 --- not very good for who
uses a black background). Also the grid color wasn't persisted in the
.eeschema file.

Sadly the grid color isn't actually set in eeschema... there is some
machinery in WinEDA_DrawFrame::LoadSettings() but I have no idea if it's
live or what (maybe it just need to be called)


--
Lorenzo Marcantonio
Logos Srl

=== modified file 'eeschema/dialogs/dialog_color_config.cpp'
--- eeschema/dialogs/dialog_color_config.cpp	2010-12-02 21:41:56 +0000
+++ eeschema/dialogs/dialog_color_config.cpp	2010-12-20 08:38:47 +0000
@@ -45,6 +45,7 @@
     { _( "Sheet" ), LAYER_SHEET },
     { _( "Sheet file name" ), LAYER_SHEETFILENAME },
     { _( "Sheet name" ), LAYER_SHEETNAME },
+    { _( "Sheet label" ), LAYER_SHEETLABEL },
     { _( "Hierarchical label" ), LAYER_HIERLABEL },
     { wxT( "" ), -1 }                           // Sentinel marking end of list.
 };

=== modified file 'eeschema/eeschema_config.cpp'
--- eeschema/eeschema_config.cpp	2010-12-08 20:12:46 +0000
+++ eeschema/eeschema_config.cpp	2010-12-20 08:40:10 +0000
@@ -534,6 +534,9 @@
     m_configSettings.push_back( new PARAM_CFG_SETCOLOR( true, wxT( "ColorErcE" ),
                                                         &g_LayerDescr.LayerColor[LAYER_ERC_ERR],
                                                         RED ) );
+    m_configSettings.push_back( new PARAM_CFG_SETCOLOR( true, wxT( "ColorGrid" ),
+                                                        &g_LayerDescr.LayerColor[LAYER_GRID],
+                                                        DARKGRAY ) );
     m_configSettings.push_back( new PARAM_CFG_BOOL( true, wxT( "PrintMonochrome" ),
                                                         &m_printMonochrome, true ) );
     m_configSettings.push_back( new PARAM_CFG_BOOL( true, wxT( "PrintSheetReferenceAndTitleBlock" ),