Pressing space in textbox submits form.

[steve-van-de-heuvel]

I cannot input a space into a textbox in a form that is within a dialog. It triggers the onclick event of dialog button.

[nightwing]

Could you please tell us more details, where is this dialog or how do you create it?

[steve-van-de-heuvel]

          formLogTime: new Form({
                colwidth: '33%',
                edge: "0 0 0 0",
                form: [
                    {
                        name: "summary",
                        title: 'Summary',
                        type: "textbox",
                    },
                    {
                        name: "value",
                        title: 'Hours (eg: 3.5 or 3:30)',
                        type: "textbox",
                        defaultValue: 0.5,
                        width: 70

                    }
                ]

            }),

[steve-van-de-heuvel]

        dialogLogTime: new Dialog('Log Time', main.consumes, {
            title: "Log Time for Project:",
            modal: true,
            allowClose : true,
            elements : [
                {
                    type: "button",
                    id: "ok",
                    color: "green",
                    caption: "OK",
                    "default": true,
                    onclick: function(){
                        var params = Project.formLogTime.toJson();
                        params.method = 'POST';
                        params.pathInfo = 'projects/'+Project.data.id+'/tracking/time/add';
                        var currentTime = new Date();
                        var recordDate = currentTime.getFullYear() + '-' + (currentTime.getMonth()+1) + '-' + currentTime.getDate();
                        params.body = {
                            "time_record[user_id]": activeCollab.user.id,
                            "time_record[value]": params.value,
                            "time_record[record_date]": recordDate,
                            "time_record[job_type_id]": params.jobTypes,
                            "time_record[summary]": params.summary,
                            "time_record[billable_status]": 1,

                            "submitted": "submitted"
                        }
                        queryApi(params, function(json) {
                            Project.dialogLogTime.hide();
                            var bubble = imports["notification.bubble"];
                            bubble.popup("Time logged successfully.", false, function(){
                            });
                        });

                    }

                }
            ]
        })
    }
    ui.addElement(Project.contextMenu);
    ui.addElement(Project.dialogLogTime);
    ui.addElement(Project.formLogTime);

    Project.dialogLogTime.on('draw', function(e){
        Project.formLogTime.attachTo(e.html);
    });

[nightwing]

Indeed there is a bug in focus handling
Here's a code snippet for testing easily from browser console

Form = app.Form
f = new Form({
    colwidth: '33%',
    edge: "0 0 0 0",
    form: [{
        name: "summary",
        title: 'Summary',
        type: "textbox",
    }, {
        name: "value",
        title: 'Hours (eg: 3.5 or 3:30)',
        type: "textbox",
        defaultValue: 0.5,
        width: 70

    }]

})
Dialog = app.Dialog
d = new Dialog('Log Time', [], {
    title: "Log Time for Project:",
    modal: true,
    allowClose: true,
    elements: [{
        type: "button",
        id: "ok",
        color: "green",
        caption: "OK",
        "default": true,
        onclick: function() {
            debugger
        }
    }]
})
d.on('draw', function(e) {
    f.attachTo(e.html);
});
d.show()

[steve-van-de-heuvel]

Thanks for confirming. Will just use form submit button for now.