Hi Piotr, I'm facing a complicated situation now, since we need to finish our App in following days, and we're running out of time, and today we have a problem with latest change in compiler about Number to int conversion for AMF dates. So sorry, but can invest time on this for now.
Just to be able to put our conversation on a hold for few days: I think there's good points in your changes, like the ones related to cleaning the code that comes from the basic Panel and that is of no use in FormItem, but in the other hand as you said we defocus on the main point. I think we need to separate things. One is "cleaning code" that is useless, other is how to get FormItem have different views and comply with EOD (Ease of development) so we have a "default view" (the current one) and by configuration the user can change from Horizontal to Stacked. Then there's another task around content layout configuration, that should be able to do easily with CSS and not making the user to create a class definition (to change type of layout , gap, and alignment ) Do you agree with this points? Can't dedicate much time for now, but I promise to work on a solution as I get time. We are very near to finish our migration, but this days are difficult for the issues happening right now on different parts that are making us to have problems to continue... thanks Carlos El mar., 29 ene. 2019 a las 22:21, Piotr Zarzycki (< piotrzarzyck...@gmail.com>) escribió: > Carlos, > > No it's not doing what I need. Let's stay with your design if you really > need it. I did my changes in order to give myself possibilities to create > StackedFormView and discover all of that problems on the road. Current > design simply doesn't allow implement StackedFormView. Maybe when you start > working some day on StackedFormView you will face the same problems. > > My changes clean up FormItemView from some unnecessary bead transferring, > give possibility to place label with required label on top of the item. > Above design do not allows you to do that at all. > > For example IFormItemContentArea - This is a Group and in above design if > you put there HGroup (cause you can) - it will be mismatch - because of > that transferring - you got in html class="jewel horizontal itemsCenter > layout vertical gap-3x3px" > > All of my changes make sure that this situation is not going to happen. I > started this thread to ask about possibilities how to manipulate some part > of Forms, but this discussion redirect into different direction. > > I will stay with my StackedFormItemView and all changes in FormItemView in > my application. > > Thanks for your thoughts, > Piotr > > > > > wt., 29 sty 2019 o 18:51 Carlos Rovira <carlosrov...@apache.org> > napisał(a): > > > Piotr, > > > > if you build current develop Jewel and TourDeJewel, you'll see that you > can > > change content layout just setting the layout at formitem level: > > > > This is vertical content: > > > > <j:FormItem label="Favorite movies (at least 2)" required="true"> > > <j:CheckBox text="The Godfather"/> > > <j:CheckBox text="Scent of a woman"/> > > <j:CheckBox text="Star Wars"/> > > </j:FormItem> > > > > > > > > This is horizontal content: > > > > <j:FormItem label="Verification code: {randomCode}" required="true"> > > <j:beads> > > <j:HorizontalLayout gap="3"/> > > </j:beads> > > <j:TextInput> > > <j:beads> > > <j:TextPrompt prompt="Enter Verification Code"/> > > <j:StringValidator validateFunction="{customValidate}"/> > > </j:beads> > > </j:TextInput> > > <j:Button localId="btn" text="send" emphasis="primary"/> > > </j:FormItem> > > > > That's what you want right? (not L and * involved here, since that will > be > > another view, ok?) > > > > So don't think there's content in your branch needed since we already > have > > the solution. > > > > Let me know if I'm missing something. > > > > > > > > > > El mar., 29 ene. 2019 a las 18:27, Piotr Zarzycki (< > > piotrzarzyck...@gmail.com>) escribió: > > > > > But you didn't merge my branch and in FormItemView where changes. How > > can I > > > in develop change content layout without changing L and Required label > ? > > > > > > wt., 29 sty 2019 o 18:23 Carlos Rovira <carlosrov...@apache.org> > > > napisał(a): > > > > > > > El mar., 29 ene. 2019 a las 17:19, Piotr Zarzycki (< > > > > piotrzarzyck...@gmail.com>) escribió: > > > > > > > > > > > > > > I didn't say anywhere that I'm going to commit any of that group to > > > > > Framework. User should be responsible for creating such Group. - > > PAYG - > > > > I'm > > > > > showing how to do that in example. > > > > > > > > > > > > > > ok, cool. But I think we need to do our best to have this easily > > > configured > > > > with what we provide. Other thing is the user tries to do something > > very > > > > different, in that case is up to them to code its and plug. > > > > > > > > > > > > > > > > > Can you say what solution are you going to try ? > > > > > > > > > > > > > > I was looking at the code dedicating more time and I think the > solution > > > is > > > > already there. > > > > I just made a commit in develop that removes in the FormExample (Tour > > de > > > > Jewel) the surrounding HGroup and VGroup, so the FormItem is capable > to > > > use > > > > the layout we setup to layout contents in both ways, that I think is > > what > > > > our users will want and is the most straight forward way to do. So > > users > > > > will want to change the layout of the content mostly, for others that > > > want > > > > to change the "outer" layout to have Stacked or other kind of global > > form > > > > item layout, that should happen with an extension of the current > > > > FormItemView to swap (while that want will be able to do what the > > actual > > > > one is capable right now). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > El mar., 29 ene. 2019 a las 16:32, Piotr Zarzycki (< > > > > > > piotrzarzyck...@gmail.com>) escribió: > > > > > > > > > > > > > By "my Group" - I'm not saying about Group component > > specifically - > > > > I'm > > > > > > > talking about component which allows you to specify that gap > for > > > > > example > > > > > > > VGroup. > > > > > > > > > > > > > > wt., 29 sty 2019 o 16:31 Piotr Zarzycki < > > piotrzarzyck...@gmail.com > > > > > > > > > > > napisał(a): > > > > > > > > > > > > > > > I know that Group doesn't have gap. I'm saying that IF I > would > > > like > > > > > to > > > > > > > > have children layed out differently - I will have to provide > my > > > own > > > > > > > Group - > > > > > > > > VGroup for example. > > > > > > > > > > > > > > > > IF I would like to have those children laying out vertically > > with > > > > > gap - > > > > > > > my > > > > > > > > Group should come with that gap. > > > > > > > > > > > > > > > > wt., 29 sty 2019 o 16:28 Carlos Rovira < > > carlosrov...@apache.org> > > > > > > > > napisał(a): > > > > > > > > > > > > > > > >> Hi Piotr, > > > > > > > >> > > > > > > > >> I think you missed my latest response or I didn't could > > > translate > > > > > the > > > > > > > main > > > > > > > >> idea: Group doesn't have gap. So "he need > > > > > > > >> to provide appropriate Group with gap - Am I right ?" is not > > > right > > > > > to > > > > > > > >> figure that. > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> El mar., 29 ene. 2019 a las 15:52, Piotr Zarzycki (< > > > > > > > >> piotrzarzyck...@gmail.com>) escribió: > > > > > > > >> > > > > > > > >> > Carlos, > > > > > > > >> > > > > > > > > >> > This one won't work. > > > > > > > >> > > > > > > > > >> > .verticalContentFormItem > > > > > > > >> > { > > > > > > > >> > IFormItemContentArea: > > > > > > > ClassReference("org.apache.royale.jewel.VGroup"); > > > > > > > >> > gap: 3; > > > > > > > >> > } > > > > > > > >> > > > > > > > > >> > Cause how that gap would be propagated to > > IFormItemContentArea > > > > ? I > > > > > > > think > > > > > > > >> > the solution is here if someone wanted to have gap between > > > > > elements > > > > > > he > > > > > > > >> need > > > > > > > >> > to provide appropriate Group with gap - Am I right ? > > > > > > > >> > > > > > > > > >> > Thanks, > > > > > > > >> > Piotr > > > > > > > >> > > > > > > > > >> > wt., 29 sty 2019 o 11:45 Carlos Rovira < > > > carlosrov...@apache.org > > > > > > > > > > > > >> > napisał(a): > > > > > > > >> > > > > > > > > >> > > Piotr, I think your problem is with having no gap > between > > > > items > > > > > > > right? > > > > > > > >> > > > > > > > > > >> > > you can get it adding "gap:3" (search for that in other > > > parts > > > > of > > > > > > > >> Jewel) > > > > > > > >> > > > > > > > > > >> > > for example: > > > > > > > >> > > > > > > > > > >> > > .verticalContentFormItem > > > > > > > >> > > { > > > > > > > >> > > IFormItemContentArea: > > > > > > > >> ClassReference("org.apache.royale.jewel.VGroup"); > > > > > > > >> > > gap: 3; > > > > > > > >> > > } > > > > > > > >> > > > > > > > > > >> > > remember to add this in sass and not in css or will be > > > > override > > > > > > when > > > > > > > >> > > generating all themes > > > > > > > >> > > > > > > > > > >> > > thanks > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > El lun., 28 ene. 2019 a las 21:03, Piotr Zarzycki (< > > > > > > > >> > > piotrzarzyck...@gmail.com>) escribió: > > > > > > > >> > > > > > > > > > >> > > > Hi Guys, > > > > > > > >> > > > > > > > > > > >> > > > I just make changes which I have mention in other > > thread. > > > > > > > >> > > > > > > > > > > >> > > > 1. I have removed code which transfers beads through > the > > > > > > > components. > > > > > > > >> > [1] > > > > > > > >> > > In > > > > > > > >> > > > the results: > > > > > > > >> > > > IBeadLayout - Is responsible for general layout of > > > FormItem > > > > > > > >> (default: > > > > > > > >> > > > HorizontalLayout) > > > > > > > >> > > > IFormItemContentArea - Is responsible for content > > > (default: > > > > > > Group) > > > > > > > >> > > > 2. I have removed IFormItemLayout from default.css > cause > > > it > > > > > > wasn't > > > > > > > >> used > > > > > > > >> > > > 3. I have adjusted ToureDeJewel to new changes. > However > > > > there > > > > > is > > > > > > > one > > > > > > > >> > > > problem which I need to get your thoughts how to > > resolve. > > > > > > > >> > > > In TourDeJewel example there is FormItem with vertical > > > > layout, > > > > > > so > > > > > > > to > > > > > > > >> > have > > > > > > > >> > > > them laying out vertically you need: > > > > > > > >> > > > > > > > > > > >> > > > 1. > > > > > > > >> > > > .verticalContentFormItem > > > > > > > >> > > > { > > > > > > > >> > > > IFormItemContentArea: > > > > > > > >> ClassReference("org.apache.royale.jewel.VGroup"); > > > > > > > >> > > > } > > > > > > > >> > > > 2. <j:FormItem label="Favorite movies (at least 2)" > > > > > className=" > > > > > > > >> > > > verticalContentFormItem" />> > > > > > > > >> > > > > > > > > > > >> > > > In the results we have https://imgur.com/a/gV8DnyS - > > The > > > > same > > > > > > is > > > > > > > >> with > > > > > > > >> > > next > > > > > > > >> > > > item which has content HGroup. > > > > > > > >> > > > > > > > > > > >> > > > Because of that I committed my changes to > > > > > > > >> feature/formitemview_cleanup. > > > > > > > >> > > > I think as a user - I will have to create my own Group > > > which > > > > > > > taking > > > > > > > >> > care > > > > > > > >> > > of > > > > > > > >> > > > that stuff - What do you think ? > > > > > > > >> > > > > > > > > > > >> > > > After my changes each part of FormItem is doing the > > right > > > > > things > > > > > > > and > > > > > > > >> > css > > > > > > > >> > > > looks like that: > > > > > > > >> > > > > > > > > > > >> > > > j|FormItem { > > > > > > > >> > > > IBeadLayout: > > > > > > > >> > > > > > > > > > > >> > > > > > > > > >> > > > > > > > > > > > > > > > > ClassReference("org.apache.royale.jewel.beads.layouts.HorizontalLayout"); > > > > > > > >> > > > IBeadView: > > > > > > > >> > > > > > > > > > > > > ClassReference("org.apache.royale.jewel.beads.views.FormItemView"); > > > > > > > >> > > > IBeadModel: > > > > > > > >> > > > > > > > > > > >> > > > > > > ClassReference("org.apache.royale.jewel.beads.models.FormItemModel"); > > > > > > > >> > > > IFormItemContentArea: > > > > > > > >> > ClassReference("org.apache.royale.jewel.Group"); > > > > > > > >> > > > } > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > [1] > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/royale-asjs/commit/2af5cc10fbe064d5176162d8c8ee54f6edd6cafb > > > > > > > >> > > > > > > > > > > >> > > > Thanks, > > > > > > > >> > > > -- > > > > > > > >> > > > > > > > > > > >> > > > Piotr Zarzycki > > > > > > > >> > > > > > > > > > > >> > > > Patreon: *https://www.patreon.com/piotrzarzycki > > > > > > > >> > > > <https://www.patreon.com/piotrzarzycki>* > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > -- > > > > > > > >> > > Carlos Rovira > > > > > > > >> > > http://about.me/carlosrovira > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > > >> > -- > > > > > > > >> > > > > > > > > >> > Piotr Zarzycki > > > > > > > >> > > > > > > > > >> > Patreon: *https://www.patreon.com/piotrzarzycki > > > > > > > >> > <https://www.patreon.com/piotrzarzycki>* > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > > >> -- > > > > > > > >> Carlos Rovira > > > > > > > >> http://about.me/carlosrovira > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > > > Piotr Zarzycki > > > > > > > > > > > > > > > > Patreon: *https://www.patreon.com/piotrzarzycki > > > > > > > > <https://www.patreon.com/piotrzarzycki>* > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > Piotr Zarzycki > > > > > > > > > > > > > > Patreon: *https://www.patreon.com/piotrzarzycki > > > > > > > <https://www.patreon.com/piotrzarzycki>* > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Carlos Rovira > > > > > > http://about.me/carlosrovira > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > Piotr Zarzycki > > > > > > > > > > Patreon: *https://www.patreon.com/piotrzarzycki > > > > > <https://www.patreon.com/piotrzarzycki>* > > > > > > > > > > > > > > > > > -- > > > > Carlos Rovira > > > > http://about.me/carlosrovira > > > > > > > > > > > > > -- > > > > > > Piotr Zarzycki > > > > > > Patreon: *https://www.patreon.com/piotrzarzycki > > > <https://www.patreon.com/piotrzarzycki>* > > > > > > > > > -- > > Carlos Rovira > > http://about.me/carlosrovira > > > > > -- > > Piotr Zarzycki > > Patreon: *https://www.patreon.com/piotrzarzycki > <https://www.patreon.com/piotrzarzycki>* > -- Carlos Rovira http://about.me/carlosrovira