Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development
Added some of the missing controls to wxSmith
seb_seb0:
Hello,
I have a crash occuring with wxBitmapComboBox (on Windows Vista).
Insert a new wxBitmapComboBox in a wxPanel.
Modify the field "number of images" to a positive value in the property grid.
I have tried to debug, but the bug did not appear during debugging.
The bug *seems* to happen in the following lines (in function wxsBitmapComboBox::OnExtraPropertyChanged, in file wxsbitmapcombobox.cpp, start at line 228)
--- Code: ---for(int i = m_iNumImages; i < iNewFields; i++) {
m_arrBitmapIds[i] = Grid->GetGrid()->AppendIn(m_idImages, NEW_IN_WXPG14X wxImageFileProperty(wxString::Format(wxT("%s %d"), sImage.c_str(), i + 1), wxPG_LABEL));
}
--- End code ---
but I have no guarantees for that. Does anybody have an idea ?
If I find a solution, I will post it of course.
Sebastien
Cryogen:
Hi Guys,
I'm way behind the discussion due to computer death. It's killing me. :-(
--- Quote from: seb_seb0 on July 20, 2010, 09:59:23 pm ---Mea culpa.
I have double-checked, and I have noticed that I have copied the file to another directory.
It is corrected, and now it works (of course).
Sorry for the disturbance - mistake was on my side.
--- End quote ---
I know. ;-)
--- Quote from: seb_seb0 on July 20, 2010, 09:59:23 pm ---Other topic:
I am approaching the end of the XPMEditor plugin (hopefully). Once I have fixed the remaining bugs, I will gladly give a hand on wxSmith.
The first thing I would like to see done is a better handling of images in wxSmith : typically your Patch 1 is a very good start.
You might be aware that rcoll has done another plugin which can be interesting to merge with wxSmith: it is called wxSmithImage. See here: http://forums.codeblocks.org/index.php/topic,12362.0.html (reply #5)
It already has code to handle:
- images lists
- embedded images in the main wxFrame / wxDialog / wxPanel code.
Let me know what you think.
--- End quote ---
Yes, I'm already well on the way to integrating parts of that into the mainline code as it adds things we need such as image list management. It's a nice piece of work. I was working on that when the machine died. After being messed about by a local hardware shop I'm now awaiting parts and a warranty repair. Until it's fixed I'm trying to set up CB portable but the box I'm using right now won't even let me run the installer without admin privileges, which of course I don't have. This is where Windows absolutely sucks. So, I'm trying other avenues.
Ciao.
Cryogen:
Hi,
--- Quote from: western1 on July 20, 2010, 11:31:52 pm ---I also noticed libwxsmith.so was built on May 27, which seems old enough to include cryo's patches up to #12 (thanks Cryo).
...
Thanks to all of you for your excellent contributions!!!
--- End quote ---
Any time. ;-)
--- Quote from: western1 on July 20, 2010, 11:31:52 pm ---Do I need to build from source and apply the diff file myself?
--- End quote ---
Yes, sir.
--- Quote from: western1 on July 20, 2010, 11:31:52 pm ---(If I need to apply the patch, does 12 include all of the previous ones?)
--- End quote ---
No. That will be stated if it happens.
Cheers.
Cryogen:
--- Quote from: MortenMacFly on July 23, 2010, 07:09:30 am ---BTW: I assume it_s tested under Linux? Because that'd be a problem for to do me ATM.
--- End quote ---
You betcha. Always. :-)
Cryogen:
--- Quote from: MortenMacFly on July 23, 2010, 09:09:24 pm ---
--- Quote from: MortenMacFly on July 23, 2010, 07:09:30 am ---Just for the record: I am currently testing the whole patch. If it works, I'll commit.
--- End quote ---
Lucky winner: Its accepted an committed to trunk. Some of the new controls I've even already used in other projects using earlier patches of yours. So I thought it's really time to provide your hard work to the community. :-)
--- End quote ---
Great, thanks.
--- Quote from: MortenMacFly on July 23, 2010, 09:09:24 pm ---Could you please update the "patchorama" now to reflect the submission and make it easier for me to see what's left?
--- End quote ---
Yes but see my PM for more.
--- Quote from: MortenMacFly on July 23, 2010, 09:09:24 pm ---BTW: The weight which you have applied to the different controls might be worth a discussion. But we will see whether there are complains on that issue after the next nightly.
--- End quote ---
Yes, as suggested some time back. It's currently purely alphabetical, which makes most sense to me although I understand that that moves apart some related controls. I think we probably need to add some tabs and group a few things that way.
Cheers.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version