Added wxWrapSizer #258

Merged
merged 1 commit into from Feb 8, 2017

Projects

None yet

2 participants

@jackburridge
Contributor

Added wxWrapSizer, didn't know what the icon should be

@jhasse

Looks good :)

+public:
+ wxObject* Create(IObject *obj, wxObject * /*parent*/)
+ {
+ wxWrapSizer *sizer = new wxWrapSizer(obj->GetPropertyAsInteger(_("orient")));
@jhasse
jhasse Dec 2, 2016 Member

Try to align the * with the type like you did with the return type of Create.

+ ticpp::Element* ExportToXrc(IObject *obj)
+ {
+ ObjectToXrcFilter xrc(obj, _("wxWrapSizer"));
+ if( obj->GetPropertyAsSize(_("minimum_size")) != wxDefaultSize ) xrc.AddProperty(_("minimum_size"), _("minsize"), XRC_TYPE_SIZE);
@jhasse
jhasse Dec 2, 2016 Member

This line is a little bit too long, better:

if( obj->GetPropertyAsSize(_("minimum_size")) != wxDefaultSize )
{
        xrc.AddProperty(_("minimum_size"), _("minsize"), XRC_TYPE_SIZE);
}
+ {
+ XrcToXfbFilter filter(xrcObj, _("wxWrapSizer"));
+ filter.AddProperty(_("minsize"), _("minsize"), XRC_TYPE_SIZE);
+ filter.AddProperty(_("orient"),_("orient"),XRC_TYPE_TEXT);
@jhasse
jhasse Dec 2, 2016 Member

Missing spaces after the ,s.

@jhasse jhasse added this to the 3.6.1 milestone Dec 14, 2016
@jhasse jhasse merged commit e769c8e into wxFormBuilder:master Feb 8, 2017

1 check passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@jhasse jhasse added a commit that referenced this pull request Feb 8, 2017
@jhasse jhasse Fix small style issues for #258 d0f3342
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment