12.0p+ ComboBoxItem pickListPlacement: "fillScreen", addUnknownValues: true broken

[Blama]

Hi Isomorphic,

please see this testcase here (v12.0p_2021-03-06, in 12.1p and 13.0d as well).
When you enter an non existing value and hit "Enter" in the ComboBoxItem marked broken, the value is not accepted.
It only works with the click on "Accept". This is not much of a problem in this testcase, but happens the same on mobile, where people use the "go" button (see screenshot), which I think resembles the "Enter" button on desktops. This is very confusing (leading to a bug report for us), as the "go" button does work as accepted when you confirm an existing value with it.

isc.DynamicForm.create({
    ID: "testForm1",
    isGroup: true,
    groupTitle: "Normal, addUnknownValues: true (default)",
    width: 500,
    fields: [
    {
        name: "filteredCombo", title: "Select Item", editorType: "ComboBoxItem",
        addUnknownValues: true,
        optionDataSource: "supplyItem",
        displayField:"itemName", valueField:"SKU",
        filterFields:["SKU", "itemName"],
        pickListWidth:300,
        pickListFields:[
            {name:"SKU"},
            {name:"itemName"}
        ]
    }
    ]
});

isc.DynamicForm.create({
    ID: "testForm2",
    isGroup: true,
    groupTitle: "pickListPlacement: 'fillScreen', addUnknownValues: true (default) <b>(broken)</b>",
    width: 500,
    fields: [
    {
        name: "filteredCombo", title: "Select Item", editorType: "ComboBoxItem",
        addUnknownValues: true,
        optionDataSource: "supplyItem",
        displayField:"itemName", valueField:"SKU",
        filterFields:["SKU", "itemName"],
        pickListPlacement: "fillScreen",
        pickListWidth:300,
        pickListFields:[
            {name:"SKU"},
            {name:"itemName"}
        ]
    }
    ]
});

isc.DynamicForm.create({
    ID: "testForm3",
    isGroup: true,
    groupTitle: "pickListPlacement: 'fillScreen', addUnknownValues: false",
    width: 500,
    fields: [
    {
        name: "filteredCombo", title: "Select Item", editorType: "ComboBoxItem",
        addUnknownValues: false,
        optionDataSource: "supplyItem",
        displayField:"itemName", valueField:"SKU",
        filterFields:["SKU", "itemName"],
        pickListPlacement: "fillScreen",
        pickListWidth:300,
        pickListFields:[
            {name:"SKU"},
            {name:"itemName"}
        ]
    }
    ]
});

isc.VLayout.create({
    membersMargin:10,
    members:[testForm1, testForm2, testForm3]
});

Best regards
Blama

[Isomorphic]

So to be clear, you're saying that your end users are expecting the "Go" button to work like "Accept" does?

In addition are you saying that you think there shouldn't be a separate "Accept" button?

[Blama]

Hi Isomorphic,

only the former. I think the Accept button makes a lot of sense.

Best regards
Blama

[Isomorphic]

We've made a fix to address this issue in the 12.1p and 13.0d branches
Please try the next nightly build dated March 20 or above

Regards
Isomorphic Software

[Blama]

Hi Isomorphic,

I can see that this is fixed in current 12.1p. It's not very important for me, but if the fix is easy, could you also fix it in 12.0p?

Best regards
Blama