combobox grid: avoid needing two clicks after re-selecting an item

'picker.hide()' hides the picker, but does not do everything to
properly keep track of the picker state in the combobox class.

This lead to a bug when we reselected an entry, we had to click the
picker again twice to open it again.

Use the 'collapse' method of the combobox instead, which does the
necessary book-keeping.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

diff --git a/src/form/ComboGrid.js b/src/form/ComboGrid.js
index ba3ce409f7f9bedfb9c5ba978fff56479f8a4b08..29c3d26bb7d9dbf73effdbc4a3a72e546319723e 100644 (file)
--- a/src/form/ComboGrid.js
+++ b/src/form/ComboGrid.js
@@ -290,7 +290,7 @@ Ext.define('Proxmox.form.ComboGrid', {
        if (!me.multiSelect) {
            picker.on('itemclick', function(sm, record) {
                if (picker.getSelection()[0] === record) {
-                   picker.hide();
+                   me.collapse();
                }
            });
        }