Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

Lost the code-completion window border

(1/2) > >>

Loaden:
After merged with trunk (scintilla branch to trunk).
Any comment?

MortenMacFly:

--- Quote from: Loaden on September 16, 2010, 05:29:26 am ---Any comment?

--- End quote ---
This is true... I didn't realise this so far... However, the modifications are actually not related to the CC Kick-In Windows at all?! Weired...?!

Jenna:
This patch fixes it for me, but this code was not changed in the update, so I don't know what has happened.


--- Code: ------ tmp/tmpFZA7Cp-meld/PlatWX.cpp
+++ home/jens/codeblocks-build/codeblocks.pure.trunk/src/sdk/wxscintilla/src/PlatWX.cpp
@@ -828,7 +828,7 @@
         SetBackgroundColour(*wxBLACK);  // for our simple border
 
         lv = new wxSCIListBox(parent, id, wxDefaultPosition, wxDefaultSize,
-                              wxLC_REPORT | wxLC_SINGLE_SEL | wxLC_NO_HEADER | wxBORDER_NONE);
+                              wxLC_REPORT | wxLC_SINGLE_SEL | wxLC_NO_HEADER | wxSIMPLE_BORDER);
         lv->SetCursor(wxCursor(wxCURSOR_ARROW));
         lv->InsertColumn(0, wxEmptyString);
         lv->InsertColumn(1, wxEmptyString);
@@ -954,7 +954,7 @@
     {
 
         lv = new wxSCIListBox(this, id, wxDefaultPosition, wxDefaultSize,
-                              wxLC_REPORT | wxLC_SINGLE_SEL | wxLC_NO_HEADER | wxNO_BORDER);
+                              wxLC_REPORT | wxLC_SINGLE_SEL | wxLC_NO_HEADER | wxSIMPLE_BORDER);
         lv->SetCursor(wxCursor(wxCURSOR_ARROW));
         lv->InsertColumn(0, wxEmptyString);
         lv->InsertColumn(1, wxEmptyString);

--- End code ---

MortenMacFly:

--- Quote from: jens on September 16, 2010, 01:12:34 pm ---This patch fixes it for me, but this code was not changed in the update, so I don't know what has happened.

--- End quote ---
Jens, you are the hero. I was looking inside the lower-level Scintilla code for long, but found nothing. Then I had to skip. I'll look into the changes again to see why this is needed suddenly... Please commit! :-)

Jenna:

--- Quote from: MortenMacFly on September 16, 2010, 06:56:13 pm ---
--- Quote from: jens on September 16, 2010, 01:12:34 pm ---This patch fixes it for me, but this code was not changed in the update, so I don't know what has happened.

--- End quote ---
Jens, you are the hero. I was looking inside the lower-level Scintilla code for long, but found nothing. Then I had to skip. I'll look into the changes again to see why this is needed suddenly... Please commit! :-)

--- End quote ---
Done (trunk r6601), I also added the ability to change fore- and background-color of CallTips ("stolen" from ScintillaBase and slightly modified).

Navigation

[0] Message Index

[#] Next page