Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development
wxSmith improvement
Cryogen:
Hi Guys,
The wxAnimationCtrl/wxMediaCtrl patch is up:
http://forums.codeblocks.org/index.php/topic,12584.new.html
Cryo.
Cryogen:
Hi Guys,
Just a quick update to keep things together.
--- Quote from: Cryogen on May 18, 2010, 08:13:43 am ---Here are some other things I'd like to see done:
-Update Codef()'s "%n" option to use wxT() rather than _T(), as that's what wxWidgets has standardised on.
-Likewise change instances of _T() in the code to wxT() or _(), as translation isn't done for some reason.
-Add an option to pass a float or double through Codef() as you can't get one through easily, currently.
--- End quote ---
I'm getting into the guts of this thing, now. It's pretty complicated, I must say. I now see that translation is available in much of the code. Some does need to be fixed, though.
Some other thoughts that have occurred:
-There are a lot of typos in the documentation that could be cleaned up.
-The doxygen file comments are missing, which prevents some things from working.
-Doxygen sees wxPropertyGrid's docs as the main doc when you try to compile wxSmith's docs. We just need to adjust things a bit to fix this.
-The author used his preferred code style, which is fine, but it might be good to change what is output by wxSmith (not the internal code as that would be mega-work) so that it produces a standard ID format and variable names like m_Var, m_pVar, etc. I believe that would bring it more into line with C::B and wx but we'd need to discuss it.
-It would be nice to prevent controls displaying on the palette if they're not compiled into the wx lib in use.
-Others that I can't remember on the spur of the moment.
These are done and tested on Win7 and Ubuntu Karmic:
http://forums.codeblocks.org/index.php/topic,12622.0.html
http://forums.codeblocks.org/index.php/topic,12677.0.html
There is definitely room for improvement and I'm aware of at least one limitation that I need to correct soon.
This where I'm at currently...
I've added most of the remaining controls from wx2.8.
I'm currently struggling with a couple of things such as image lists for things like wxSimpleHtmlListBox and how to manage cell allocation for wxGridBagSizer.
Once I've sorted these I'll release another patch.
I've started adding doxygen file comments to the files I change and cleaning up the control palette tabs as I go. (Reorganising the tabs is trivial so it's easy to change it if we decide on another arrangement later.)
I've been updating the list at http://wiki.codeblocks.org/index.php?title=Comparison_of_wxSmith_features progressively.
I would then like to add the remaining managed windows. I've already made a start on this.
Next, I propose to do a clean up of the typos and such and release a separate patch, independent of my other changes, for that.
Finally, I'd like to look at wx2.9 and start on the controls that that introduces.
That should do for a while. ;-)
Cryo.
MortenMacFly:
--- Quote from: Cryogen on June 08, 2010, 06:42:12 pm ----Doxygen sees wxPropertyGrid's docs as the main doc when you try to compile wxSmith's docs.
--- End quote ---
This will change in the near future. wxPropGrid will be updated and be part of the core (as a DLL).
Cryogen:
--- Quote from: MortenMacFly on June 10, 2010, 07:16:07 am ---
--- Quote from: Cryogen on June 08, 2010, 06:42:12 pm ----Doxygen sees wxPropertyGrid's docs as the main doc when you try to compile wxSmith's docs.
--- End quote ---
This will change in the near future. wxPropGrid will be updated and be part of the core (as a DLL).
--- End quote ---
That's good, thanks. That reminds me to update the list below... ;-)
Cryogen:
--- Quote from: Cryogen on June 08, 2010, 06:42:12 pm ---
--- Quote from: Cryogen on May 18, 2010, 08:13:43 am ----Update Codef()'s "%n" option to use wxT() rather than _T(), as that's what wxWidgets has standardised on.
-Likewise change instances of _T() in the code to wxT() or _(), as translation isn't done for some reason. [This is largely wrong but we should check translations anyway]
-Add an option to pass a float or double through Codef() as you can't get one through easily, currently. [Partially done]
--- End quote ---
-There are a lot of typos in the documentation that could be cleaned up.
-The doxygen file comments are missing, which prevents some things from working.
-Doxygen sees wxPropertyGrid's docs as the main doc when you try to compile wxSmith's docs. We just need to adjust things a bit to fix this. [Fixed in C::B SVN 6354 with PropGrid 1.4]
-The author used his preferred code style, which is fine, but it might be good to change what is output by wxSmith (not the internal code as that would be mega-work) so that it produces a standard ID format and variable names like m_Var, m_pVar, etc. I believe that would bring it more into line with C::B and wx but we'd need to discuss it.
-It would be nice to prevent controls displaying on the palette if they're not compiled into the wx lib in use.
-Others that I can't remember on the spur of the moment.
--- End quote ---
-Update wxPropGrid to the latest. [Fixed in C::B SVN 6354 with PropGrid 1.4]
-Fix the awful screen flashing that occurs every time the editor display updates.
I've now got past my initial problem with image lists but the current solution only uses images from files, i.e. wxImageFileProperty, as there's no way to get wxSmith's image editor into the grid when you are dynamically adding and removing fields. We need to extend the image editor component to allow it to dynamically add and remove editors. So...
-Extend the image editor to allow dynamic addition and removal of image fields. [Fixed in my patch 18]
-The same for the colour editor.
I'm also working on a demo app. that I use as a testbed and which will provide examples of how to use the controls I add, based somewhat on the wx samples.
Cheers.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version