Draggable / formeo

Drag & Drop Form Builder
http://draggable.github.io/formeo/
MIT License
530 stars 198 forks source link

The preview does not match the options. #306

Open rodrigoGA opened 3 years ago

rodrigoGA commented 3 years ago

I have found several bugs when editing a form.

1- open the sample application https://draggable.github.io/formeo/ 2- create a Radio Group 3- change the name of some option, for example "Radio 2" to "new radio 2" 4- press the edit button radio Group / options BUG: The preview does not match the options. 5- remove option 1 and remove option 3 6- create 2 new options 7- press edit button. BUG The preview does not match the options.

The fields that have options have the same bug

dougauerbach commented 3 years ago

Can you try clicking into the settings for each of the radio or checkbox options, rather than typing the value directly into label? This is the workaround we have had to use to deal with this bug.

rodrigoGA commented 3 years ago

I understand, but it is still a bug. Maybe it would be better if editing is disabled in the preview to avoid problems.

dougauerbach commented 3 years ago

I agree. The contenteditable=true attribute is driving this.

We added CSS to make the controls look non-editable. Suboptimal, of course, but it doesn't invite the user to find the bug.

CSS:

    .formeo.formeo-editor .f-radio [contenteditable]:after,
    .formeo.formeo-editor .f-checkbox [contenteditable]:after {
        border-bottom: none;
    }

On Tue, Apr 20, 2021 at 10:56 AM Rodrigo @.***> wrote:

I understand, but it is still a bug. Maybe it would be better if editing is disabled in the preview to avoid problems.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Draggable/formeo/issues/306#issuecomment-823483730, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFN6MWWXM3K675UXRXYM5M3TJW547ANCNFSM4XXFJ3YQ .