Hi, Morten, I don't understand this sentence, what does the "hack in a First place" mean?
I mean as a first commit it would be fair enough to just remove the hack with the de-selection. Not removing logic with changing images from coloured to b/w.
Ok, follow your advice, the patch becomes a lot simple, just remove the hack of de-selection. See below:
Index: E:/code/cb/cb_trunk_sf/src/src/editorconfigurationdlg.cpp
===================================================================
--- E:/code/cb/cb_trunk_sf/src/src/editorconfigurationdlg.cpp (revision 8972)
+++ E:/code/cb/cb_trunk_sf/src/src/editorconfigurationdlg.cpp (working copy)
@@ -309,7 +309,7 @@
void EditorConfigurationDlg::AddPluginPanels()
{
const wxString base = _T("images/settings/");
- const wxString noimg = _T("images/settings/generic-plugin");
+ const wxString noimg = _T("images/settings/generic-plugin"); //for those plugins who does not supply icons
wxListbook* lb = XRCCTRL(*this, "nbMain", wxListbook);
// get all configuration panels which are about the editor.
@@ -346,12 +346,6 @@
lb->SetPageImage(i, (i * 2) + (sel == (int)i ? 0 : 1));
}
- // the selection colour is ruining the on/off effect,
- // so make sure no item is selected ;)
- // (only if we have icons showing)
- if (GetSettingsIconsStyle(lb->GetListView()) != sisNoIcons)
- lb->GetListView()->Select(sel, false);
-
// update the page title
wxString label = lb->GetPageText(sel);
// replace any stray & with && because label makes it an underscore
BTW: I see the "selection ruining " only affect on the non-transparent part of the icon. See the image shot below, the colorful icon of codecompletion is selected under Windows, but not the full icon has ruined compared with the third icon screen shot in my original post.