On 03/04/2014 05:56 PM, Adam Misnyovszki wrote:

----- Original Message -----
From: "Adam Misnyovszki" <amisn...@redhat.com>
To: "Petr Vobornik" <pvobo...@redhat.com>
Cc: "freeipa-devel" <freeipa-devel@redhat.com>
Sent: Tuesday, March 4, 2014 5:27:21 PM
Subject: Re: [Freeipa-devel] [PATCH] 545 webui: Don't act on keyboard events 
which originated in, different dialog

----- Original Message -----
From: "Petr Vobornik" <pvobo...@redhat.com>
To: "freeipa-devel" <freeipa-devel@redhat.com>
Sent: Tuesday, February 25, 2014 2:20:11 PM
Subject: [Freeipa-devel] [PATCH] 545 webui: Don't act on keyboard events
which originated in, different dialog

Fixes issue when:
1. 2 dialogs are opened
2. top dialog's close button is focused
3. user presses enter to execute 'close' action
4. dialog is immediately closed (enter key is still pressed)
5. second dialog automatically receives focus (it's top dialog now)
6. user releases the key
7. second dialog reacts to keyup event - which is by default
confirmation mixin's confirm event
8. UNDESIRED behavior occurs

Now confirmation mixin remembers which keys were pressed and released
and reacts only to those which originated there.

Petr Vobornik

However, we should keep in mind, that there is an issue, when the ENTER key
is pressed for a long time(ie more time than a keydown event is fired), the
dialog starts to "blink". In my oppinion, this later behaviour is a wontfix,
because firstly it is not easy to handle,
secondly because this problem exists in mostly all softwares, thirdly this is 
not the scope of the ticket.
Sorry for the amendment, zimbra doesn't work quite well.

Pushed to master: 9b540ef21864f8da822bd92d58385339147e773d


Freeipa-devel mailing list

Reply via email to