Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Development => Topic started by: Loaden on September 16, 2010, 05:29:26 am

Title: Lost the code-completion window border
Post by: Loaden on September 16, 2010, 05:29:26 am
After merged with trunk (scintilla branch to trunk).
Any comment?
Title: Re: Lost the code-completion window border
Post by: MortenMacFly on September 16, 2010, 07:04:11 am
Any comment?
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...?!
Title: Re: Lost the code-completion window border
Post by: Jenna 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.

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);
Title: Re: Lost the code-completion window border
Post by: MortenMacFly on September 16, 2010, 06:56:13 pm
This patch fixes it for me, but this code was not changed in the update, so I don't know what has happened.
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! :-)
Title: Re: Lost the code-completion window border
Post by: Jenna on September 16, 2010, 10:26:11 pm
This patch fixes it for me, but this code was not changed in the update, so I don't know what has happened.
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! :-)
Done (trunk r6601), I also added the ability to change fore- and background-color of CallTips ("stolen" from ScintillaBase and slightly modified).
Title: Re: Lost the code-completion window border
Post by: MortenMacFly on September 17, 2010, 06:51:49 am
Done (trunk r6601), I also added the ability to change fore- and background-color of CallTips ("stolen" from ScintillaBase and slightly modified).
Thanks. I had a look again at your changes. In fact, isn't it weird that the border had been there until now? I mean: If I got your changes right, then it was set implicitly to wxBORDER_NONE, was it?! So the actual question is why this was visible in the past... :shock:

However, it's back to "normal" now luckily.
Title: Re: Lost the code-completion window border
Post by: Loaden on September 17, 2010, 07:12:23 am
Nice! works well now.( I just merged from trunk to cc branch manual. )
@Morten, could you merge the changes from trunk to cc branch?
Thanks!