I do not know the cause of the bug - it comes however from the Bitmap Editor patch.
I tried to debug it, but it did not happen during a debugging session (probably a memory initialisation problem / release)
I provide here a single patch for: Dialogs, Pickers, Media + Animation, Hyperlink, Missing controls. Done against latest SVN
+ wxGLCanvas patched for wx2.9.x (#if #endif construction)
+ wxGrid patch (added 1 event handler wxGRID_CELL_CHANGED)
This patch does not produce the bug.
It will be hopefully easier to apply in the trunk.
I am also missing some icons for (probably a file missing / mispelled):
wxHtmlEasyPrinting
wxDialupManager
wxContextHelpButton
wxListView
wxRichTextCtrl
wxSearchCtrl
wxSimpleHtmlListBox
wxTreebook
Thank you for the good work !
Sebastien
I am also missing some icons for (probably a file missing / mispelled):
wxHtmlEasyPrinting
wxDialupManager
wxContextHelpButton
wxListView
wxRichTextCtrl
wxSearchCtrl
wxSimpleHtmlListBox
wxTreebook
Did you download the image pack below? It sounds like you missed it. :wink:
I am also missing some icons for (probably a file missing / mispelled):
Did you download the image pack below? It sounds like you missed it. :wink:
Yes I did, and I checked that I had all the icons in my folders "share\..."
I will double check anyway, and let you know.
Adds:Just for the record: I am currently testing the whole patch. If it works, I'll commit.
wxBitmapComboBox
wxContextHelpButton
wxDialUpManager
wxHtmlEasyPrinting
wxListView
wxRichTextCtrl
wxSearchCtrl
wxSimpleHtmlListBox
wxTreebook
Just for the record: I am currently testing the whole patch. If it works, I'll commit.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. :-)
Some information about the patch I send you:@seb_seb0: Could you please verify that this matches your expectations / experiences? Hopefully I didn't miss something...
Some information about the patch I send you:@seb_seb0: Could you please verify that this matches your expectations / experiences? Hopefully I didn't miss something...
..\..\..\sdk\wxpropgrid\include/wx/propgrid/propdev.h:18: warning: type attributes ignored after type is already defined
1 - icons are missing in wxSmith, for:Fixed that in SVN.
2 - the wxSmithAUI cbp project file contains 2 targets : Windows & Unix. It leads to a warning message when recompiling the workspace. Is it possible to split the project in 2 files (1 for each target), and add only the Windows target to the workspace ? (and same for UNIX)I actually like this idea of having just one project file very much, although it throws an error (actually it's a warning only). I think it's a nice "tech-demo" to explain the "platform" feature of C::B for project files. So I'd rather leave it like it is.
Fixed that in SVN.2 - the wxSmithAUI cbp project file contains 2 targets : Windows & Unix. It leads to a warning message when recompiling the workspace. Is it possible to split the project in 2 files (1 for each target), and add only the Windows target to the workspace ? (and same for UNIX)I actually like this idea of having just one project file very much, although it throws an error (actually it's a warning only). I think it's a nice "tech-demo" to explain the "platform" feature of C::B for project files. So I'd rather leave it like it is.
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));
}
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.
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 (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.
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!!!
Do I need to build from source and apply the diff file myself?
(If I need to apply the patch, does 12 include all of the previous ones?)
BTW: I assume it_s tested under Linux? Because that'd be a problem for to do me ATM.
Just for the record: I am currently testing the whole patch. If it works, I'll commit.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. :-)
Could you please update the "patchorama" now to reflect the submission and make it easier for me to see what's left?
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.
2 - the wxSmithAUI cbp project file contains 2 targets : Windows & Unix. It leads to a warning message when recompiling the workspace. Is it possible to split the project in 2 files (1 for each target), and add only the Windows target to the workspace ? (and same for UNIX)
3 - I have many times the same warning regarding wxPropGrid:CodeWould it be possible to remove it ? (probably a #define to do somewhere, or in the compilation line)...\..\..\sdk\wxpropgrid\include/wx/propgrid/propdev.h:18: warning: type attributes ignored after type is already defined
I actually like this idea of having just one project file very much, although it throws an error (actually it's a warning only). I think it's a nice "tech-demo" to explain the "platform" feature of C::B for project files. So I'd rather leave it like it is.
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)Codebut I have no guarantees for that. Does anybody have an idea ?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));
}
If I find a solution, I will post it of course.
Sebastien