User forums > Using Code::Blocks

Configure dialog icon issue when clicked

<< < (4/4)

ollydbg:

--- Quote from: MortenMacFly on April 09, 2013, 09:59:03 pm ---Just the hack in a First place would be fine and does Not affect 3rd Party plugins at all.

--- End quote ---
Hi, Morten, I don't understand this sentence, what does the "hack in a First place" mean?

MortenMacFly:

--- Quote from: ollydbg on April 10, 2013, 04:20:25 pm ---Hi, Morten, I don't understand this sentence, what does the "hack in a First place" mean?

--- End quote ---
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.

ollydbg:

--- Quote from: MortenMacFly on April 10, 2013, 05:38:48 pm ---
--- Quote from: ollydbg on April 10, 2013, 04:20:25 pm ---Hi, Morten, I don't understand this sentence, what does the "hack in a First place" mean?

--- End quote ---
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.

--- End quote ---
Ok, follow your advice, the patch becomes a lot simple, just remove the hack of de-selection. See below:

--- Code: ---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

--- End code ---

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.

MortenMacFly:

--- Quote from: ollydbg on April 12, 2013, 03:29:15 am ---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.


--- End quote ---
Even better - go ahead then! :-)

ollydbg:
Done in rev8985, thanks everyone.

Navigation

[0] Message Index

[*] Previous page