wxSmith improvement

[seb_seb0]

Hello,

I do not know if you have seen my post here:
http://forums.codeblocks.org/index.php/topic,13243.msg89078.html#msg89078

I have sent a patch against SVN 6570 which solves the problem with wxsMyColourPropertyClass.
It is a 10 lines patches, and it solved the problem for the colour combobox in wxSmith.

Let me know if that helps you, or if I misunderstood completly the last posts.

Sebastien

[Jenna]

Hello,

I do not know if you have seen my post here:
http://forums.codeblocks.org/index.php/topic,13243.msg89078.html#msg89078

I have sent a patch against SVN 6570 which solves the problem with wxsMyColourPropertyClass.
It is a 10 lines patches, and it solved the problem for the colour combobox in wxSmith.

Let me know if that helps you, or if I misunderstood completly the last posts.

Sebastien

It shows the colourboxes, but the rest is not working (can not chose a colour).
There seem to be api-changes, so some of the choices do not longer work.
For the coulours we can use the properties of wxPropGrid, I don't know if there is a need to still use the customcolourpropoerty.
I did not find a solution for the editors, but don't have much time to look into it (not at home the next two or three days).

[seb_seb0]

Hello,

I do not know if you have seen my post here:
http://forums.codeblocks.org/index.php/topic,13243.msg89078.html#msg89078

I have sent a patch against SVN 6570 which solves the problem with wxsMyColourPropertyClass.
It is a 10 lines patches, and it solved the problem for the colour combobox in wxSmith.

Let me know if that helps you, or if I misunderstood completly the last posts.

Sebastien

It shows the colourboxes, but the rest is not working (can not chose a colour).
There seem to be api-changes, so some of the choices do not longer work.
For the coulours we can use the properties of wxPropGrid, I don't know if there is a need to still use the customcolourpropoerty.
I did not find a solution for the editors, but don't have much time to look into it (not at home the next two or three days).

Unfortunately you are right.
If I find some time, I will try a bit deeper. I will let you know if I find something.

Sebastien

[Cryogen]

Hello,

I do not know if you have seen my post here:
http://forums.codeblocks.org/index.php/topic,13243.msg89078.html#msg89078

I have sent a patch against SVN 6570 which solves the problem with wxsMyColourPropertyClass.
It is a 10 lines patches, and it solved the problem for the colour combobox in wxSmith.

Let me know if that helps you, or if I misunderstood completly the last posts.

Sebastien

Nice catch, thanks Seb. That's one step forward, at least. I'm still looking as well.

[Cryogen]

 Hi Guys,

Two things. I've done some more work on wxSmith's problems, specifically the colour property. The main problem seems to be that

Code

int type = wxEnumPropertyClass::DoGetValue().GetLong();

in wxsMyColourPropertyClass::OnEvent() usually returns 0. I would have said "always" except that during the last run I saw the value 33 a few times, for unknown reasons. Either way, it always ends up failing to match any of the conditionals and therefore, we get no action. I've been unable to trace why, other than that it falls through to wxPGProperty::DoGetValue(). I can't see how the value is set or why it's not getting the right value.

I also tried building a new property based on the 1.4 docs, but it behaves the same way as soon as I start using wxEnumPropertyClass.

I think we need to try to resurrect it if we can because it adds some features over the default wxSystemColourProperty, such as the "default" entry. One option might be to steal that and modify it.

There seems to be some sort of refresh problem, in general, that stops the text appearing for the other properties but, again, where and why is anyone's guess.

If whomever did the upgrade has a better knowledge of wxSmith's internals, perhaps it should go back to them to find out what didn't upgrade as intended?

The other thing is that I received a call 15 mins ago saying that I've finally got a real job. So, from Monday, I will have much less time to work on C::B. Reality bites, huh? ;-)
I'll still be around, though.

Cheers.

[Jenna]

I attach a patch that seems to fix the issue for wxsColourProperty, it's not deeply tested (and not tested at all on windows and mac), but seems to work.

Please other devs (and users of course) please test and give feedback.

I will look into the other issues (wxListbox and others) later.

The patch is against trunk (r6595), but I think it works with all other branches too.

EDIT:
attachment removed, actual patch see http://forums.codeblocks.org/index.php/topic,12362.msg89758.html#msg89758

[Jenna]

After some testing, I found that the patch is just a starting point not a solution, it works for existing elements, but crashes if a new one with colourproperty is added.

[Jenna]

After some testing, I found that the patch is just a starting point not a solution, it works for existing elements, but crashes if a new one with colourproperty is added.

New version (a little larger), but this time it seems to work correctly.

It's against trunk and not (yet) tested on windows.

Any feedback is welcome.

EDIT:
attachment removed, actual patch see http://forums.codeblocks.org/index.php/topic,12362.msg89758.html#msg89758

[MortenMacFly]

Any feedback is welcome.

I had applied the first one, too - no serious issues so far. I'll try this one, too. If you hear nothing all is fine. :-)

[Jenna]

Small update to my previous patch (build/link-fix for windows):

either use the patch attached here or remove WXDLLIMPEXP_PG from class-declaration of wxsMyColourPropertyClass in wxscolourproperty.cpp:124.

I remove the older patches.

[Cryogen]

 [EDIT](nonsense removed)

My bad, the patch didn't, apparently. Ergh. It's working well, now.  :oops:

Win7 and SVN 6575.

Bye.

[Jenna]

Just committed the fix for the colourproperty.

[Jenna]

Just committed the fix for the colourproperty.

The other issues (related to wxCustomEditorProperty) should be fixed in trunk now (svn r6441) and will be fixed in cc- and dbg-branch afetr the next merge.

[Cryogen]

 Hey Jens,

The other issues (related to wxCustomEditorProperty) should be fixed in trunk now (svn r6441) and will be fixed in cc- and dbg-branch afetr the next merge.

Many thanks for fixing this. It's all working properly now, and better thanks to the enhancements you made. It also fixed another issue I was seeing that must have been related.

Cheers,

 G.

[Cryogen]

 Hi Guys,

I've added the rest of wxSmithImage's editors to wxSmith. Patch 21 in the patchorama. Details are here: http://forums.codeblocks.org/index.php/topic,13213.0.html.
I'll update the demo app over the next few days. I think the tree editor will allow me to complete the treebook control, now, as well, which is nice.

Ciao.