News:

As usual while waiting for the next release - don't forget to check the nightly builds in the forum.

Main Menu

wxSmith development

Started by byo, May 26, 2005, 02:38:44 AM

Previous topic - Next topic

cyberkoa

Quote from: byo
Hello, everyone. It's been a while since I've been here because of these sleepless books-full nights and terrible exams-like days. Just give me few more days and I'll be back :D.

Cyberoka: BIG THX for Youre work. Cna You send me the source ? And maybe some description of problems You have with this widget.
and don't worry - I'm CVS-beginner too (started using it after gaining access to CodeBlocks' CVS). Buty If You don't mind I would like to work now like mandrav and rigkg22 at the beginning of their cooperation - You could send me the sourcee and I put this into CVS :). Hope it will change soon - I just need to get back to this project and prepare for new Coder :). And now ... argh, back to books :?
any kind of cooperation that will be fine for me , in fact I hv learned a lot from reading the wxSmith code, still have a lot to learn :)

Let me do some final cleaning of the source code before post it up .
One thing need to mention , my Code::Blocks CVS is not the latest update because the latest update revamp the interface which affect some plugin coding.  My Code::Blocks CVS version should be somewhere on early of June.

I shall post it after going back home , and write some description on the problem that I encounter.

cyberkoa

Here are the files
http://us.f1f.yahoofs.com/bc/96491ee4_m418d23c5/bc/wxSmith-Combobox.zip?bfA3juCBjIkFen5d

Basically, it includes 2 new files , wxscombobox.h and wxscombobox.cpp , and others modified files.

One thing I have left out is the releasing memory usage of the wxArrayString that used in the wxscombobox , which I think should put in the destructor of each widgets class.
I do not change the original macro in defwidgets.h that have the blank destructor because I think this changes involve the change of the engine should decided by byo.

Feel free to modify it if you feel those code is not correct or uncomplete.

I only coded the XML load and XML save , other function like AddProperty is not coded yet.
Thanks.

rickg22

Quote from: byoCyberoka: BIG THX for Youre work. Cna You send me the source ?

Quote from: CyberokaHere are the files (link)

:roll: Someone needs to be given access to CVS... :lol:

cyberkoa

Finally , I manage to come in this forum   :o

Ok , I have finished the code for wxListBox , and I have made icons for wxCombobox and wxListBox , which is the new feature introduced by byo in the latest CVS.

Let me summarize my changes and enhancement
1.  Add in wxListBox
2.  Modified a macro in wxdefwidget.h , to add in destructors evDestroy() function. This is to release the memory used by wxArrayString.
3.  I have made icons for wxCombox and wxListBox , I have no sense of art dunno looks ok or not  :oops:

My discovery and suggestion
1.  The default selection for wxComboBox and wxListBox , since they are defined as Int property , there is not upper bound checking.  Could it be better if we can design in such a way that we can choose to show the property in different form such as a combo box to limit the default selection to number of choices.


Finally, how can I commit the changes to CVS using Tortoise CVS?   :oops:

I am not familiar with CVS , I just want to update to the wxSmith branch .

mandrav

QuoteFinally, how can I commit the changes to CVS using Tortoise CVS?
First, add in CVS any new files you 've created (if any). To do this, select the new files, right-click and select "Add" from the tortoise menu.
Then, just right-click in the wxSmith top-folder (in a white space) and select "Commit". Fill the message log describing your changes and press OK :)

Yiannis.
Be patient!
This bug will be fixed soon...

cyberkoa

I have a problem to commit the changes because Tortoise CVS is using anonymous to log on every time I tried to commit .  

 I have tried a few way including recheck out the CVS using my sourceforge ID ..
or do I need to recheck out a fresh copy first ? :(

rickg22

cyberkoa: Yes, you need to Checkout a fresh copy with your sourceforge account.

cyberkoa

I still having problem to commit to CVS ,
And I am rushing back to my home country Malaysia tomorrow morning , therefore, I have send my source to byo and hope that he can help to commit the changes.

Since I will be at my home country the next two weeks, I can't help much in wxSmith at these moment.  

See you all then .. :)

byo

Quote from: cyberkoaI still having problem to commit to CVS ,
And I am rushing back to my home country Malaysia tomorrow morning , therefore, I have send my source to byo and hope that he can help to commit the changes.

Since I will be at my home country the next two weeks, I can't help much in wxSmith at these moment.  

See you all then .. :)

Got new sources, will put it into CVS as soon as possible  :)

BTW: First attempt to generate code on-the-fly was made :) There are still some bugs but it seems to be working :D

byo

As we can see, C::B has been branched. And because wxSmith hasn't got any release yet, I've decided to develop it under HEAD branch. It means switching to wxWidgets 2.6.1, and what follows, fighting many incompatibilities between different wxWidgts versions inside wxSmith. Hope that change to wx 2.6 is permament ;)

mandrav

Quote from: byoHope that change to wx 2.6 is permament ;)
That's the point of branching :)

The VERSION_1_0 branch will only accept bug-fixes. Feel free to experiment in the main trunk (HEAD).

Yiannis.
Be patient!
This bug will be fixed soon...

byo

I have problem with one bug after migrating to docking panels.
New Notebook page added to Manager for wxSmith uses wxSplitterWindow, but  after startup andafter docking Manager, sash position is set to 0 or sth like that (top window and sash are hidden). I used additional wxPanel with one sizer as background window for page but it didn't work out. The only way to see both panels is to use SetSashGravity and manipulate with size of manager.

Any ideas ?

rickg22

Try memorizing the splitter position and restore it on docking (i just hope there's an event for that :P ). For startup, use a configmanager variable.

mandrav

Quote from: byoI have problem with one bug after migrating to docking panels.
New Notebook page added to Manager for wxSmith uses wxSplitterWindow, but  after startup andafter docking Manager, sash position is set to 0 or sth like that (top window and sash are hidden). I used additional wxPanel with one sizer as background window for page but it didn't work out. The only way to see both panels is to use SetSashGravity and manipulate with size of manager.

Any ideas ?
That was the exact erroneous behaviour I got when used wxDockit with the project manager which also uses a wxSplitterWindow.
The sash would move to position -22 hiding the open files list and the sash itself.
This turned out to be a bug of wxDockit, which I fixed and documented. When the layout was updated, wxDockit would first launch the wxEVT_LAYOUT_CHANGE and then actually perform the layout. I 've corrected it in the wxDockit sources in our CVS and everything started working fine.

I haven't checked wxSmith's code for a while, but are you using a wxSplitterWindow directly?
You might want to use a wxSplitPanel, created by Rick (in xtra_classes.h). See projectmanager.cpp for how it is used...

[edit]
Also, check out EditorManager::RefreshOpenFilesTree() which is called when the layout changes (it could be called by OnSize). This is what keeps the sash in sync after layout.
[/edit]

Yiannis.
Be patient!
This bug will be fixed soon...

cyberkoa

I have extended my home leave for extra 2 weeks, still at Malaysia now.

byo , I think you already fix the bugs you mentioned right ? since I want to see the new version of CB , I hv downloaded the latest CVS and compile the latest version . I hv tried CVS dated 26/7/2005 still got problem but 27/7/2005 it seems like ok.

mandrav, I hv downloaded CB rc-1 and try , it works fine , but when I finished compile wxSmith , and try to run , it give me msg "Another CB process is running" , I hv tried to restart my PC , it give the same msg also.

so , I used back the final-beta version .
I am using winXP , wxwidgets-2.6.1