-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] form layout is error prone and intimidating (dnd fields) #750
base: master
Are you sure you want to change the base?
Conversation
@ebrehault hi. Is there any progress of the IDE work? One thing thats unintuitive currently is that you have to type the id first and select it before adding a new field. This is obviously something that will change with the IDE, but if its going to take some time I can probably do a quick fix here. |
…n't yet disable tinymces table controls.
I fixed few bugs in the ide branch before I left to vacations, I guess I need to review it one more time, and then I will release it. |
This will also fix - #626 |
@displacedaussie @ebrehault The current inserting into tinymce is here https://github.com/plomino/Plomino/blob/master/ide/app/editors/tiny-mce.component.ts#L82. It currently has a bug that it doesn't insert at the cursor. Something like this might fix it - http://blog.squadedit.com/tinymce-and-cursor-position. Since angular is in typescript we either need to rewrite the code that inserts field etc, or turn it into a function that can be called from TS, or, since we have to do an ajax call to get the widget anyway, we could do it all serverside. |
User problem
Currently layout is done with markup and character styles. Less has the following downsides
Options
Use tinymce's noneditable mode and inset representative html for fields, actions and subforms. html doesn't have to be perfect but should try to show what the final result will look like.
Labels, displayfields, hidewhens and caches can also use noneditable to take up less space and be less error prone.
Here is an example of WIP for fields
TODO: