diff options
| author | Pavel Vomacka <pvomacka@redhat.com> | 2017-01-04 12:21:57 +0100 |
|---|---|---|
| committer | Martin Babinsky <mbabinsk@redhat.com> | 2017-02-17 17:50:32 +0100 |
| commit | 1a96e7f9e737941480436269668ccde0a50395f9 (patch) | |
| tree | 439d1a000cb38d735f23e4d3420251a9c10519fa /install/ui | |
| parent | d05d1115e409962fee3576a4bfc5cecfacef4fd3 (diff) | |
| download | freeipa-1a96e7f9e737941480436269668ccde0a50395f9.tar.gz freeipa-1a96e7f9e737941480436269668ccde0a50395f9.tar.xz freeipa-1a96e7f9e737941480436269668ccde0a50395f9.zip | |
WebUI: add default on_cancel function in adder_dialog
Adder dialog is mixed with confirmation_mixin. That mixin calls on_cancel method
when closing dialog using ESC key. Previously the on_cancel method
was not defined, therefore dialog was not correctly closed. This was the root
cause of the bug, that adder dialog cannot be opened after closing it using ESC.
Now the default function for on_cancel is dialog.close. So dialog
is correctly closed.
https://fedorahosted.org/freeipa/ticket/6388
Reviewed-By: Petr Vobornik <pvoborni@redhat.com>
Reviewed-By: Martin Basti <mbasti@redhat.com>
Diffstat (limited to 'install/ui')
| -rw-r--r-- | install/ui/src/freeipa/add.js | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/install/ui/src/freeipa/add.js b/install/ui/src/freeipa/add.js index 622108526..b93bd3463 100644 --- a/install/ui/src/freeipa/add.js +++ b/install/ui/src/freeipa/add.js @@ -36,6 +36,8 @@ IPA.entity_adder_dialog = function(spec) { var that = IPA.dialog(spec); + that.on_cancel = that.close; + IPA.confirm_mixin().apply(that); /** @property {string} method="add" API method for add command */ |